Opened 9 years ago
Closed 8 years ago
#35571 closed defect (bug) (fixed)
Update TinyMCE and jQuery UI button styles
Reported by: | iseulde | Owned by: | iseulde |
---|---|---|---|
Milestone: | 4.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | has-patch |
Focuses: | Cc: |
Description
Even though there's a big note on top of the buttons.css
since [27494], it has been ignored in all the following commits.
We need to update those files, or maybe consider moving the classes to the buttons.css
file.
@helen what do you think?
Attachments (4)
Change History (24)
#2
@
9 years ago
Yes, but I guess buttons.css
can be made a dependency. We just need to add extra classes to that file. Happy to make a patch to review.
This ticket was mentioned in Slack in #core-editor by iseulde. View the logs.
9 years ago
#6
@
9 years ago
#7
@
9 years ago
- Owner set to iseulde
- Status changed from new to assigned
@iseulde Can you make sure the styles get corrected for 4.5?
#8
@
9 years ago
I used this Diff as a guide to see what changes have been made since the note was added to the top of buttons.css
This ticket was mentioned in Slack in #core by chriscct7. View the logs.
9 years ago
This ticket was mentioned in Slack in #core by chriscct7. View the logs.
9 years ago
This ticket was mentioned in Slack in #core by mike. View the logs.
9 years ago
#12
@
8 years ago
@iseulde @helen: Can you please review this? Wondering if it's still on track to land for 4.5.
#15
@
8 years ago
So we'd rather patch the current files again instead of moving them to buttons.css?
#16
@
8 years ago
@iseulde Whatever you think is best here. If you'd like to go the dependency route, feel free to either patch/commit that way, or punt.
I don't mind if we commit the current approach (bringing the files up to date), then handle it differently for 4.6, if that makes the most sense.
#17
@
8 years ago
In late beta, yes, let's patch a file and not go messing with what depends on which.
I don't recall why those styles were left in separate files, but I imagine it had something to do with the styles being loaded in different contexts. How much needs to be synced and are there situations where
buttons.css
isn't loaded where the other ones are?