Closed
Bug 591582
Opened 14 years ago
Closed 14 years ago
Style Panel CSS properties are limited to hard coded values
Categories
(DevTools :: General, defect)
DevTools
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jwalker, Assigned: miker)
References
Details
Attachments
(1 file, 1 obsolete file)
58.00 KB,
application/vnd.ms-excel
|
Details |
CssHtmlTree contains a list of StyleGroupView items which group CSS properties. This list could easily get out of date.
(dolske) I think you really need a way to dynamically get a list of all known-to-Gecko CSS properties (which I believe are common to all elements?), group the ones this code knows about, and then plop everything else into this "Other" group. Alternatively, write a test that looks at these tables and fails when an new (unknown) property is added.
The problem here is that no one is going to remember to modify this code when new CSS properties are added in the future. So, either a test is needed to ensure people know to do that, or new properties should automatically show up somewhere generic (at which point users would presumably notice and file a bug to clean up the UI).
Reporter | ||
Updated•14 years ago
|
Updated•14 years ago
|
Whiteboard: [kd4b7]
Updated•14 years ago
|
Whiteboard: [kd4b7]
Assignee | ||
Comment 1•14 years ago
|
||
Complete list of used and unused official and unofficial CSS properties sorted
into groups.
The shorthand properties still need to be removed from the list.
A large amount of these properties can be found in
/layout/style/test/property_database.js but others were gathered from CSS 1, 2 & 3
standards websites.
Using this method we can categorize most properties. It would still be good to put any remaining CSS properties under the "Effects and Other" category.
Looking at the number of attributes in "Effects and Other" maybe we should split this into more categories?
Assignee | ||
Updated•14 years ago
|
Summary: csshtmltree needs a way to keep up with CSS changes → Style Panel CSS properties are limited to hard coded values
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → mratcliffe
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•14 years ago
|
||
Assignee | ||
Comment 4•14 years ago
|
||
Comment on attachment 530312 [details] [diff] [review]
Unsupported properties to be moved to "other" group
This patch has been added to the patch in bug 582596 to speed up the review process
Attachment #530312 -
Attachment is obsolete: true
Assignee | ||
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•