Skip to content

DOC-2609: Resizing images by typing into the input didn't update the …#3582

Merged
abhinavgandham merged 9 commits intofeature/7.6.1/DOC-2609from
feature/7.6.1/DOC-2609_TINY-11669
Jan 21, 2025
Merged

DOC-2609: Resizing images by typing into the input didn't update the …#3582
abhinavgandham merged 9 commits intofeature/7.6.1/DOC-2609from
feature/7.6.1/DOC-2609_TINY-11669

Conversation

@abhinavgandham
Copy link

@abhinavgandham abhinavgandham commented Jan 17, 2025

… attribute correctly.

Ticket: DOC-2609

Site: Staging branch

Changes:

Pre-checks:

  • Branch prefixed with feature/<version>/, hotfix/<version>/, staging/<version>/, or release/<version>/.

Review:

  • Documentation Team Lead has reviewed

@abhinavgandham abhinavgandham requested review from MitchC1999 and removed request for a team January 17, 2025 08:00
@abhinavgandham abhinavgandham added the upcoming release Documentation for features currently under development/QA label Jan 17, 2025
@abhinavgandham abhinavgandham added this to the TinyMCE 7.6.1 milestone Jan 17, 2025
@kemister85
Copy link
Contributor

@abhinavgandham You will need to find another name for mce-data as this is used internally only.


This occurred because the resize function was triggered while the selection was elsewhere, due to a debounce delay between the input and the resize operation. As a result, the resize operation was performed on the current selection instead of the intended image.

This led to other editor components that rely on the `style` attribute accidentally reading outdated values, and the resize not being applied correctly since the selection was not on the image.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, this Jira ended having 2 different issues and probably my description was kinda confusing.

the first problem (the one that interfere with the the other editor component) was that updating the width and height via input we didn't update correctly the data-mce-style so some other element, like some plugin, took a non updated value

the second one was about the timeout.

in the Jira documentation the 2 cases are indicated in each section via numbers 1 or 2 when you read 1. [something] that something is referring to the first case and if there is a 2 is referred to the second case.

If you want we can have a call tomorrow morning, I didn't manage this Jira well and there is a little bit of confusion in that documentation :/

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lorenzo-pomili Yeah sure

@lorenzo-pomili lorenzo-pomili self-requested a review January 21, 2025 08:09
Copy link
Contributor

@kemister85 kemister85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

abhinavgandham and others added 2 commits January 21, 2025 15:27
Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>
@abhinavgandham abhinavgandham merged commit b273113 into feature/7.6.1/DOC-2609 Jan 21, 2025
4 checks passed
@abhinavgandham abhinavgandham deleted the feature/7.6.1/DOC-2609_TINY-11669 branch January 21, 2025 10:01
kemister85 added a commit that referenced this pull request Jan 22, 2025
#3571)

* DOC-2609: TinyMCE 7.6.1 Release Documentation and Community Changelog.

* DOC-2609: The mentions menu without items was not closed when pressin… (#3578)

* DOC-2609: The mentions menu without items was not closed when pressing Space or Enter key.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: The mentions dropdown remained visible even after the menti… (#3579)

* DOC-2609: The mentions dropdown remained visible even after the mention text was deleted in quick succession.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Toggle sidebar shortcut remained active when the editor was… (#3580)

* DOC-2609: Toggle sidebar shortcut remained active when the editor was disabled.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Table resize bars was not properly aligned for inline edito… (#3581)

* DOC-2609: Table resize bars was not properly aligned for inline editors inside scrollable containers.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Images with existing srcset were not handled correctly. (#3583)

* DOC-2609: Opening a notification when the toolbar is at the bottom threw an error. (#3586)

* DOC-2609: Opening a notification when the toolbar is at the bottom threw an error.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Text input was prevented in form elements in the contents of the editor. (#3585)

* DOC-2609: Text input was prevented in form elements in the contents of the editor.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Error messages were not rendered correctly (#3584)

* DOC-2609: Error messages were not rendered correctly

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* DOC-2609: Resizing images by typing into the input didn't update the … (#3582)

* DOC-2609: Resizing images by typing into the input didn't update the  attribute correctly.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Added ticket number to entry.

* Improved wording and removed the reference of the attribute's name.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Cleaned up release docs and added security fix and self-hosted server side docs.

* Updated changelog

* Fixed up changelog to only include titles for core.

* fixed up docs entry

* API bump and copy edits.

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>
kemister85 added a commit that referenced this pull request Jan 22, 2025
* DOC-2610: TinyMCE 7.6.1 Release Documentation.

* DOC-2609: TinyMCE 7.6.1 Release Documentation and Community Changelog. (#3571)

* DOC-2609: TinyMCE 7.6.1 Release Documentation and Community Changelog.

* DOC-2609: The mentions menu without items was not closed when pressin… (#3578)

* DOC-2609: The mentions menu without items was not closed when pressing Space or Enter key.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: The mentions dropdown remained visible even after the menti… (#3579)

* DOC-2609: The mentions dropdown remained visible even after the mention text was deleted in quick succession.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Toggle sidebar shortcut remained active when the editor was… (#3580)

* DOC-2609: Toggle sidebar shortcut remained active when the editor was disabled.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Table resize bars was not properly aligned for inline edito… (#3581)

* DOC-2609: Table resize bars was not properly aligned for inline editors inside scrollable containers.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Images with existing srcset were not handled correctly. (#3583)

* DOC-2609: Opening a notification when the toolbar is at the bottom threw an error. (#3586)

* DOC-2609: Opening a notification when the toolbar is at the bottom threw an error.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Text input was prevented in form elements in the contents of the editor. (#3585)

* DOC-2609: Text input was prevented in form elements in the contents of the editor.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* DOC-2609: Error messages were not rendered correctly (#3584)

* DOC-2609: Error messages were not rendered correctly

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* DOC-2609: Resizing images by typing into the input didn't update the … (#3582)

* DOC-2609: Resizing images by typing into the input didn't update the  attribute correctly.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Added ticket number to entry.

* Improved wording and removed the reference of the attribute's name.

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

* Update modules/ROOT/pages/7.6.1-release-notes.adoc

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

* Cleaned up release docs and added security fix and self-hosted server side docs.

* Updated changelog

* Fixed up changelog to only include titles for core.

* fixed up docs entry

* API bump and copy edits.

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>

---------

Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

upcoming release Documentation for features currently under development/QA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants