User tests: Successful: Unsuccessful:
Pull Request resolves #39060
PR fixes the rotate tab behavior in media Manager where the manual angle input gets reset to 0 after applying rotation.
The fix keeps the user entered value in the angle field after rotation is applid.
Note:
This PR focuses on the reported bug #39060 value reset to 0.
There is also a broader UX behavior where manual rotation is triggered by change/blur events. That means if the value is unchangeed, no new rotation action is triggered. To keep this bug fix focused, I suggest handling that separately as a follow-up improvement.
For that a possible way(I can think of) is adding an explicit apply action(button) for manual angle input. IDK if that is to be considered as an feature - that is likely an enhancement and can be discussed in a separate issue/PR.
Also for the case of using arrows with cursor that also needs attention.
Administrator > content > Media.
use any image in edit mode.
go to the Rotate tab.
In the Angle field, type a value (for example 44) and click anywhere outside the field.
Verify:
The image rotates but the manual Angle input field is reset to 0
After entering a manual angle and applying rotation, the image rotates and the angle field keeps the entered value.
Please select:
Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | JavaScript Repository NPM Change |
I have tested this item ✅ successfully on 614254e
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
NPM Resource Changed
bug
PR-5.4-dev
|
||
RTC
| Labels |
Added:
RTC
|
||
✅ Final test before merge with JBT
npm run build:js| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-04-01 13:27:24 |
| Closed_By | ⇒ | muhme |
Thank you very much @krishnagandhicode for your contribution. Thanks to @adarshdubey03 and @exlemor for testing.
Thanks for the merge, and thanks to everyone who helped test : )
I have tested this item ✅ successfully on 614254e
I have tested it successfully. Thanks @krishnagandhicode!
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47462.