Summary: | font sizes in design mode | ||
---|---|---|---|
Product: | WebKit | Reporter: | Philip Tucker <ptucker> |
Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED LATER | ||
Severity: | Normal | CC: | ahmad.saleem792, apavlov, justin.garcia, ptucker, rniwa, wenson_hsieh |
Priority: | P2 | ||
Version: | 525.x (Safari 3.1) | ||
Hardware: | Mac | ||
OS: | OS X 10.5 |
Description
Philip Tucker
2008-08-07 13:35:07 PDT
Hey it looks like we implemented this wrong. It looks like all of the browsers use a font tag with size="n", even if they've been told to styleWithCSS. I think following test case do provide bit of coverage for fontsize via execCommand but not when they are defined as "x-small" etc. Link - https://github.com/WebKit/WebKit/blob/main/LayoutTests/editing/execCommand/query-font-size.html But if I take the test case and make it JSfiddle as below: https://jsfiddle.net/1z9etd8k/1/ In Line 92 to 94, I added these "x-small" etc. and got output value of 3 in Safari 15.5 on macOS 12.4, which is same as Chrome Canary 105. But Firefox Nightly 103, does not produce any output for these new additions. Further, Firefox also not produce any output for Manual CSS font-size tests. I might be doing it completely wrong but I think updating the test and raising this as part of InterOp 2022 for "execCommand" could be good. If I am testing incorrectly or screwed-up on test case update then ignore my comment. Thanks! Google docs no longer uses content editable. We can consider better spec'ing fontSize but let's not reuse this bug for that. |