Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#17441 closed defect (bug) (fixed)

Out of date Browswer Message contains "minimize" icon.

Reported by: hakre Owned by: markjaquith
Milestone: 3.2 Priority: normal
Severity: trivial Version: 3.2
Component: General Keywords: has-patch
Focuses: Cc:


Top-Right corner, looks like a implementation glitch on the DOM/CSS part of the feature. That message box has no min/max functionality.

Related: #17323

Attachments (1)

17441.diff (1.5 KB) - added by aaroncampbell 7 years ago.

Download all attachments as: .zip

Change History (8)

#1 follow-up: @duck_
7 years ago

It can be minimized. The problem is that the browser logo and link to vendor is positioned over the arrow. You can still minimize by clicking to the right of the arrow or elsewhere in the title bar to the left of the logo.

#2 @hakre
7 years ago

As you write it, exactly! The link of the image covers the button in part so it's hard to use it anylonger.

The link of the browswer picture could get a title for a smaller improvement over this. The Browser logo could be set as the background of the browser-icon div and the href could be boxed and put over it w/o interfeering that button any longer. Just as a quick idea.

#3 @aaroncampbell
7 years ago

I forgot about this. It was a problem I noticed while writing it, but at the time the plan was to no longer have the ability to minimize widgets. Since we kept that ability we probably need to come up with some kind of fix.

7 years ago

#4 @aaroncampbell
7 years ago

  • Keywords has-patch added

The patch checks the id of the meta box. If it's 'dashboard_browser_nag' then it doesn't echo the handlediv (the arrow), and it returns from the click event before making any changes.

#5 @markjaquith
7 years ago

  • Owner set to markjaquith
  • Resolution set to fixed
  • Status changed from new to closed

In [17952]:

Prevent the browser-out-of-date dashboard nag from being minimized and hide minimization arrow. props aaroncampbell. fixes #17441. see #17323.

#6 @ocean90
7 years ago

  • Milestone changed from Awaiting Review to 3.2

#7 in reply to: ↑ 1 @nemgavekort
6 years ago

  • Cc nemgavekort added


Last edited 6 years ago by nacin (previous) (diff)
Note: See TracTickets for help on using tickets.