Opened 10 years ago

Closed 9 years ago

#11905 closed Bug (fixed)

editor#resize event should pass dimensions in data

Reported by: Olek Nowodziński Owned by: Artur Delura
Priority: Normal Milestone: CKEditor 4.5.0 Beta
Component: General Version:
Keywords: Cc:

Description

At the moment, there's no simple way to save editor dimensions (those passed to editor.resize()) when user resizes the editor. It also means that there's no way to restore them, i.e. if user re-visits the page.

Following the question on SO.

Change History (10)

comment:1 Changed 10 years ago by Piotrek Koszuliński

Milestone: CKEditor 4.5.0
Status: newconfirmed

comment:2 Changed 10 years ago by Artur Delura

Owner: set to Artur Delura
Status: confirmedassigned

comment:3 Changed 10 years ago by Artur Delura

Status: assignedreview

Changes and tests in branch:t/11905. Also reported #12357.

Use cases:

  • rezise by code.
  • resize by user interface (right bottom handler).
  • maximize plugin.
  • window resize while CKEditor is maximised.
  • collapse / expand toolbar.
Version 2, edited 10 years ago by Artur Delura (previous) (next) (diff)

comment:4 Changed 9 years ago by Piotr Jasiun

Status: reviewreview_failed

comment:5 in reply to:  4 Changed 9 years ago by Artur Delura

Replying to pjasiun:

I commented them because they fails, they are new ones. I reported ticket:12357 for them as well. I don't see nothing wrong here. But instead of adding commented assersions I could also create a ticket branch #12357 and put them there.

comment:6 Changed 9 years ago by Artur Delura

Status: review_failedreview

Changes in branch:t/11905.

comment:7 Changed 9 years ago by Piotr Jasiun

Status: reviewreview_failed
  • Finish button in the manual test does not work, because 1.js file does not contain bender.test method. If manual test contains testName.js file it is the automatic part of the test which will be run when manual part of the test is done (this is why manual test with testName.js has Finish button instead of Pass). If you just want to execute some javascript in your tests put it in the HTML file or create _assets folder.
  • do not use localStorage in your test, every test should be independent, use variable instead,
  • move manual test to the resize plugin folder.

comment:8 in reply to:  7 Changed 9 years ago by Artur Delura

Status: review_failedreview

Done everything. Changes and tests in branch:t/11905.

comment:9 Changed 9 years ago by Piotr Jasiun

I pushed 2 small changed to tests and majorized branch, git:6cdf588.

comment:10 Changed 9 years ago by Piotr Jasiun

Resolution: fixed
Status: reviewclosed
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy