Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#38917 closed defect (bug) (fixed)

Paste As Text alert displays full-width

Reported by: mor10's profile mor10 Owned by: azaozz's profile azaozz
Milestone: 4.7 Priority: normal
Severity: normal Version: 4.7
Component: TinyMCE Keywords: has-patch commit reviewed
Focuses: Cc:

Description

4.7-beta4-39340

When clicking Paste as Text button for the first time, the alert that shows up has incomplete styling, spanning the full width of the screen rather than hovering over the content (see screenshot).

https://cldup.com/rj9Jx7Y6eX.png

Attachments (3)

mce-notice-1.png (38.1 KB) - added by azaozz 8 years ago.
mce-notice-2.png (22.6 KB) - added by azaozz 8 years ago.
38917.patch (400 bytes) - added by azaozz 8 years ago.

Download all attachments as: .zip

Change History (16)

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


8 years ago

#2 @iseulde
8 years ago

  • Milestone changed from Awaiting Review to Future Release

Looks like something goes wrong when TinyMCE calculates the position.

#3 @mor10
8 years ago

For reference, used to look like this:

https://cldup.com/dNTgsDy_7q.png

New alert should at least limit its footprint to something similar.

#4 @karmatosed
8 years ago

I can replicate this and can't see all the message:

https://cldup.com/KkaMllCq5d.png

It can get fixed by removing the left CSS position and adding a 50% width (probably best there's some device adjustments though):

https://cldup.com/ubj3Ree8RD.png

Last edited 8 years ago by karmatosed (previous) (diff)

#5 follow-up: @karmatosed
8 years ago

  • Keywords needs-patch added
  • Milestone changed from Future Release to 4.7

Tentatively suggesting this needs fixing for 4.7 as it's a bug that impacts the experience a lot.

Last edited 8 years ago by karmatosed (previous) (diff)

#6 @azaozz
8 years ago

How about we center the notice by percentage, 10% on each side looks good?

@azaozz
8 years ago

@azaozz
8 years ago

@azaozz
8 years ago

#7 @azaozz
8 years ago

  • Keywords has-patch added; needs-patch removed

#8 in reply to: ↑ 5 ; follow-up: @helen
8 years ago

Replying to karmatosed:

Tentatively suggesting this needs fixing for 4.7 as it's a bug that impacts the experience a lot.

Just for future reference, slating something for 4.7 during RC should indicate that it's either a serious bug in a new feature or a fairly significant regression from 4.6. I think this falls under the latter, but should be clear about that.

#9 @iseulde
8 years ago

  • Keywords commit added

Looks good as a quick fix while we wait for an upstream fix.

#10 in reply to: ↑ 8 @azaozz
8 years ago

  • Keywords reviewed added

Replying to helen:

Yeah, it is a CSS related regression that happens specifically on that screen. Works properly in the TinyMCE demo. We didn't notice it earlier as this message is shown only for new users.

#11 @azaozz
8 years ago

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

In 39367:

TinyMCE: fix the styling of notices generated by the editor UI.

Props: mor10, karmatosed, azaozz.
Fixes #38917.

#12 @azaozz
8 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopen for 4.7.

#13 @azaozz
8 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 39501:

TinyMCE: fix the styling of notices generated by the editor UI.

Props: mor10, karmatosed, azaozz.
Fixes #38917 for 4.7.

Note: See TracTickets for help on using tickets.