? ? NPM Resource Changed Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
14 Nov 2021

To the best of my knowledge and testing ability these classes are no longer used and can be safely removed. It looks like they were missed during the numerous refactoring of the atum template.

When testing either confirm they are not used by code review and/or check that the ... menu is still working as desired on smaller screens

avatar brianteeman brianteeman - open - 14 Nov 2021
avatar brianteeman brianteeman - change - 14 Nov 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Nov 2021
Category Administration Templates (admin) NPM Change
avatar khu5h1 khu5h1 - test_item - 14 Nov 2021 - Tested successfully
avatar khu5h1
khu5h1 - comment - 14 Nov 2021

I have tested this item successfully on 693e008

On Code Review and checked menu is still working as desired.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36032.

avatar Shubhamverma2796
Shubhamverma2796 - comment - 14 Nov 2021

I was testing the menu on different screens, and this is what I've found for "Galaxy fold",
Screenshot (278)
:
The above image shows that "20" is not visible clearly; it is getting overlapped.

Ps: Menu is working as desired for other screen sizes

avatar brianteeman
brianteeman - comment - 14 Nov 2021

Is it the same before this PR - I suspect it is - therefore it is completely unrelated to this PR and you should create a new issue for that

avatar Shubhamverma2796
Shubhamverma2796 - comment - 14 Nov 2021

Okay, I will open a new issue for this.

avatar Shubhamverma2796 Shubhamverma2796 - test_item - 14 Nov 2021 - Tested successfully
avatar Shubhamverma2796
Shubhamverma2796 - comment - 14 Nov 2021

I have tested this item successfully on 693e008


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36032.

avatar richard67 richard67 - change - 14 Nov 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 14 Nov 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36032.

avatar pritam825 pritam825 - test_item - 17 Nov 2021 - Tested successfully
avatar pritam825
pritam825 - comment - 17 Nov 2021

I have tested this item successfully on 693e008


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36032.

avatar wilsonge wilsonge - change - 28 Nov 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-11-28 17:16:21
Closed_By wilsonge
Labels Added: ? ? NPM Resource Changed
avatar wilsonge wilsonge - close - 28 Nov 2021
avatar wilsonge wilsonge - merge - 28 Nov 2021
avatar wilsonge
wilsonge - comment - 28 Nov 2021

Thanks!

avatar brianteeman
brianteeman - comment - 28 Nov 2021

Thanks

Add a Comment

Login with GitHub to post a comment