Right now width and height are intentionally text fields. Previously I've shunned only pixel integer width/height attributes (#2602124: Make video player responsive by default, hide not applicable width/height options) because they screwed up my ability to do responsive. Now that this is supported natively, is this a problem still?
Would someone conceivably care about using values like "50%" or "50em"? Making these pixels could clarify the UI a fair bit.
Make them pixels only.
Decide and patch.
Extra field suffix and description in the formatter settings.
Inability to do non-pixel units.
Change from a string to an integer in formatter settings.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 2719399-pixel-width-height-7.patch | 6.37 KB | sam152 |
| #3 | 2719399-pixel-width-height-3.patch | 5.59 KB | sam152 |
Comments
Comment #2
sam152 commentedComment #3
sam152 commentedComment #7
sam152 commentedComment #9
sam152 commentedI think because the values can still be set programatically in exceptional circumstances, this is fine for the 90% case.
Comment #11
sam152 commented