Opened 6 years ago

Last modified 6 years ago

#9784 review Bug

Move selection change listener that fires contentDirChange to core

Reported by: Piotrek Koszuliński Owned by: Piotrek Koszuliński
Priority: Normal Milestone:
Component: General Version: 4.0.1
Keywords: Cc:

Description

Rationale - this event is used by two plugins (and after proposed change, by 3).

Change History (1)

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

Owner: set to Piotrek Koszuliński
Status: newreview

Pushed t/9784 based on #9706 (t/9706b) with each it shares first commit (introduction of elementPath#direction).

Listener on contentDirChanged added in toolbar plugin was part of original fix proposed in #9706, but I don't know why. It should be investigated if it makes sense - maybe some code may be removed after adding that.

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