Make WordPress Core

Opened 11 years ago

Closed 10 years ago

#35571 closed defect (bug) (fixed)

Update TinyMCE and jQuery UI button styles

Reported by: iseulde Owned by: iseulde
Priority: normal Milestone: 4.5
Component: General Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

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)

35571.patch (62.1 KB ) - added by liljimmi 10 years ago.
Updated jquery-ui-dialog.css and editor.css to match changes in buttons.css
35571-2.patch (2.8 KB ) - added by liljimmi 10 years ago.
New patch without minified files
35571.2.patch (3.7 KB ) - added by iseulde 10 years ago.
35571.3.patch (8.0 KB ) - added by iseulde 10 years ago.

Download all attachments as: .zip

Change History (24)

#1 @helen
11 years ago

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?

#2 @iseulde
11 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.

#3 @iseulde
11 years ago

#35612 was marked as a duplicate.

This ticket was mentioned in Slack in #core-editor by iseulde. View the logs.


11 years ago

#5 @afercia
11 years ago

Related: #35604.

#7 @jorbin
10 years ago

  • Owner set to iseulde
  • Status newassigned

@iseulde Can you make sure the styles get corrected for 4.5?

@liljimmi
10 years ago

Updated jquery-ui-dialog.css and editor.css to match changes in buttons.css

#8 @liljimmi
10 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.


10 years ago

@liljimmi
10 years ago

New patch without minified files

This ticket was mentioned in Slack in #core by chriscct7. View the logs.


10 years ago

This ticket was mentioned in Slack in #core by mike. View the logs.


10 years ago

#12 @kirasong
10 years ago

@iseulde @helen: Can you please review this? Wondering if it's still on track to land for 4.5.

#13 @iseulde
10 years ago

Sure, if it's okay, will do this tomorrow morning (Europe).

#14 @kirasong
10 years ago

+1

Thanks! :D

#15 @iseulde
10 years ago

So we'd rather patch the current files again instead of moving them to buttons.css?

#16 @kirasong
10 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 @helen
10 years ago

In late beta, yes, let's patch a file and not go messing with what depends on which.

This ticket was mentioned in Slack in #core by mike. View the logs.


10 years ago

@iseulde
10 years ago

@iseulde
10 years ago

#19 @iseulde
10 years ago

  • Keywords has-patch added; needs-patch removed

Good for now?

#20 @iseulde
10 years ago

  • Resolutionfixed
  • Status assignedclosed

In 37076:

Update TinyMCE and jQuery UI button styles

Part props liljimmi.
Fixes #35571.

Note: See TracTickets for help on using tickets.