WordPress.org

Make WordPress Core

Opened 4 years ago

Last modified 4 years ago

#11365 assigned defect (bug)

press-this.php resize could be a little taller

Reported by: netweblogic Owned by: azaozz
Milestone: Future Release Priority: low
Severity: trivial Version: 2.8.5
Component: Press This Keywords: has-patch
Focuses: Cc:

Description

The press this pop-out window ends up having a scroll bar (Tested on Opera, FF and Chrome) because the window height is slightly too small.

I found that changing /wp-admin/press-this.php line 422 to :

window.resizeTo(720,600);

Solves the problem.

Attachments (1)

11365.patch (491 bytes) - added by Utkarsh 4 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 azaozz4 years ago

  • Milestone changed from 2.9 to 3.0

Utkarsh4 years ago

comment:2 scribu4 years ago

  • Component changed from General to Press This
  • Keywords has-patch added; press this removed
  • Owner set to noel

comment:3 Utkarsh4 years ago

  • Cc admin@… added

comment:4 hakre4 years ago

Was able to reproduce but it came to my attention that this does not happen all the times. This is most importantly related to a user setting, the height of the visual editor. It can be resized, If I make is smaller it fits into the default window and the patch woldn't be necessary.

Therefore I suggest the patch takes the actual size of the visual editor into account and not set the height to a fixed default height. What do you think Utkarsh?

comment:5 dd324 years ago

Having a long category list will also cause the scrollbars to appear.

comment:6 dd324 years ago

  • Type changed from enhancement to defect (bug)

changing to defect, as the entire functionality is not always shown due to window size.

comment:7 nacin4 years ago

  • Milestone changed from 3.0 to Future Release
  • Owner changed from noel to azaozz
  • Priority changed from normal to low
  • Status changed from new to assigned

TinyMCE uses its most recent height also in press this (stored in a cookie), and that's why you're sometimes getting scrollbars. We'd need to retrieve it and properly calculate what the height of the window should be.

Moving to future, lowering the priority, assigning to azaozz for when he returns.

Note: See TracTickets for help on using tickets.