WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#22018 closed defect (bug) (fixed)

Unexpected "Save Changes" button in "old" media uploader

Reported by: TobiasBg Owned by: nacin
Priority: normal Milestone: 3.5
Component: UI Version: 3.5
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

When clicking on the "Insert/Upload" button (next to the new/temporary "Beta Media" button) on any post edit screen, I have a "Save Changes" button right above the "Add media files from your computer" heading on the "From Computer" tab, that has not been there in WP 3.4.2, and that doesn't really make sense until something was uploaded.

Attachments (1)

22018.diff​ (472 bytes ) - added by helenyhou 14 years ago.

Download all attachments as: .zip

Change History (7)

#1 @TobiasBg
14 years ago

Further investigation: The button (coming from ​here) does have a class "hidden", so likely some CSS that should actually hide the button is gone.

#2 @SergeyBiryukov
14 years ago

  • Component General → Media
  • Milestone Awaiting Review → 3.5

@helenyhou
14 years ago

#3 @helenyhou
14 years ago

  • Component Media → UI
  • Keywords has-patch added

.button has display: inline-block, which is overriding the display: none for .hidden. Not sure what the best thing to do is - .button.hidden maybe? 22018.diff​ for that.

#4 @sabreuse
14 years ago

Patch works for me

#5 @lessbloat
14 years ago

  • Keywords commit added

+1 to 22018.diff.

#6 @nacin
14 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In 22440:

Allow the .hidden class to hide a button. props helenyhou. fixes #22018.

Note: See TracTickets for help on using tickets.