Opened 7 years ago

Closed 7 years ago

#9712 closed Bug (fixed)

Context menu open destories editor focus

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 4.0.1
Component: Core : Focus Version: 4.0
Keywords: HasTest Cc:

Description

  1. open inlineall sample
  2. Click in one editable, open context menu
  3. Click in another editable, open context menu
  • So far good
  1. Go back to open context menu in the first editable.
  • Actual: Context menu opens up with editor focus moved to the second editable

Change History (8)

comment:1 Changed 7 years ago by Garry Yao

Owner: set to Garry Yao
Status: newreview

Opened e7ddc4e for review.

comment:2 Changed 7 years ago by Garry Yao

Milestone: CKEditor 4.0.1

This one desires 4.0 milestone, otherwise immediately after the release.

comment:3 Changed 7 years ago by Garry Yao

Keywords: HasTest added

Opened test t/9712.

comment:4 Changed 7 years ago by Piotrek Koszuliński

Status: reviewreview_failed

Clean tests run on FF and Chrome, but two reds on IE9: http://ckeditor4.t/dt/plugins/font/font.html

I confirm that context menu issue is fixed on t/9712.

comment:5 Changed 7 years ago by Piotrek Koszuliński

The same strangely failing tests I saw on #9360.

comment:6 Changed 7 years ago by Garry Yao

Status: review_failedreview

WFM for the above test in IE9/Win7.

comment:7 Changed 7 years ago by Piotrek Koszuliński

Status: reviewreview_passed

Ok, today tests are passing. Perhaps my IE9 had a bad day.

comment:8 Changed 7 years ago by Piotrek Koszuliński

Resolution: fixed
Status: review_passedclosed

Masterised git:5c1f1d6

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy