Setting label fontlist and lineheight |
||||
Issue descriptionI've noticed numerous cases in the code where people have had to call SetFontList() followed by SetLineHeight(), including the file I am currently working in (SearchResultTileItemView). Would be beneficial/a good idea to make a method in label that does both?
,
Aug 3
I haven't looked for instances of this, but offhand, I don't think that really makes sense. The line height should be adjusted automatically for the font list. If the automatic adjustment isn't giving the right value, we should fix that. I'm not actively work on RenderText now, but I'll help advise/review. Please triage this yourself.
,
Aug 3
Need some clarification: Do you mean that the line height should be adjusted automatically whenever a font list is set?
,
Aug 22
|
||||
►
Sign in to add a comment |
||||
Comment 1 by oxyflush@chromium.org
, Aug 3