WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#28389 closed defect (bug) (fixed)

Undo doesn't work properly after resizing an image several times

Reported by: azaozz Owned by: azaozz
Priority: normal Milestone: 4.0
Component: TinyMCE Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Seems caused by the wpeditimage plugin. It changes the image tag after resizing that triggers another undo level.

Attachments (1)

28389.patch​ (375 bytes ) - added by iseulde 12 years ago.

Download all attachments as: .zip

Change History (6)

#1 @azaozz
12 years ago

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

In 28614:

TinyMCE: fix problems with undo/redo after resizing an image several times. Fixes #28389.

#2 @azaozz
12 years ago

In 28615:

Regex should be non-greedy, see #28389

@iseulde
12 years ago

#3 @iseulde
12 years ago

  • Keywords has-patch added
  • Resolution fixed
  • Status closed → reopened

We should remove the margin from the toolbar since some themes add a margin on both sides of a p, causing the toolbar to be displayed a lot lower.

#4 @azaozz
12 years ago

We should remove the margin from the toolbar...

Right. In theory we can continue to use a <div> as the wrapper, however to be able to remove it easily with a regex, it shouldn't have any nested divs. Using a <p> implies no nested block tags.

#5 @azaozz
12 years ago

  • Resolution → fixed
  • Status reopened → closed

In 28624:

TinyMCE: remove margins from the image toolbar wrapper, props avryl, fixes #28389

Note: See TracTickets for help on using tickets.