Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The operations links "edit", "delete", "enable/disable", "translate", "revert", "clone", etc in the rules and components overview pages don't have any classes, making them very hard to theme without javascript hacks.
I propose adding a simple class with the name of the operation to each link.
Comment | File | Size | Author |
---|---|---|---|
#3 | pretty-rules-ui.jpg | 57.62 KB | alexweber |
#1 | rules-operations-link-class-1655534-1.patch | 3.65 KB | alexweber |
Comments
Comment #1
alexweber CreditAttribution: alexweber commentedSimple patch attached to add classes to the operations links.
Comment #2
mitchell CreditAttribution: mitchell commented@klausi: what do you think?
@alexweber: just out of curiosity, could you please post screenshots of the theming you did?
Comment #3
alexweber CreditAttribution: alexweber commented@mitchell: sure, screenshot attached! (It's using the Fubik admin theme)
@klausi: the patch is really minimal, I just added a class to each link so when can tell which one does what :) Adding styles like in the image using those links is entirely up to the admin theme.
Comment #4
mitchell CreditAttribution: mitchell commented@alexweber: thank you; it is really pretty!! :) I also created #1733362: Add icons for rule editing screen as a follow up.
> Adding styles [that use] those links is entirely up to the admin theme.
I think this approach is an improvement to the 6.x implementation. This way all Rules needs is to define CSS class names, and then the icons can fit it with their own specific focus / metaphor.
To support the default admin theme, Seven, perhaps we could collaborate with Iconizer or some other Libraries API - type - icon project.
Comment #5
alexweber CreditAttribution: alexweber commented@mitchell, awesome! Thanks for the support on this!
@klausi, can you consider this please?
Comment #6
mitchell CreditAttribution: mitchell commentedComment #7
klausiCommitted, thanks!