Feature RTC NPM Resource Changed PR-5.1-dev Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
7 Oct 2023

Summary of Changes

Use Dialog for Shortcut overview, instead of BS modal.

Testing Instructions

Run npm install
Check shortcut overview works as before (j + x shortcut).

Actual result BEFORE applying this Pull Request

Works

Expected result AFTER applying this Pull Request

Works

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:
  • No documentation changes for docs.joomla.org needed
  • Pull Request link for manual.joomla.org:
  • No documentation changes for manual.joomla.org needed

Reference:

avatar Fedik Fedik - open - 7 Oct 2023
avatar Fedik Fedik - change - 7 Oct 2023
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Oct 2023
Category JavaScript Repository NPM Change Front End Plugins
avatar Fedik Fedik - change - 6 Nov 2023
Labels Added: Feature NPM Resource Changed PR-5.1-dev
avatar JonasAtZwetschke JonasAtZwetschke - test_item - 14 Nov 2023 - Tested successfully
avatar JonasAtZwetschke
JonasAtZwetschke - comment - 14 Nov 2023

I have tested this item ✅ successfully on 7ff0791


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

avatar viocassel viocassel - test_item - 14 Nov 2023 - Tested successfully
avatar viocassel
viocassel - comment - 14 Nov 2023

I have tested this item ✅ successfully on 7ff0791


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

avatar Quy Quy - change - 14 Nov 2023
Status Pending Ready to Commit
avatar Quy
Quy - comment - 14 Nov 2023

RTC


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

avatar LadySolveig LadySolveig - change - 16 Dec 2023
Labels Added: RTC
avatar LadySolveig LadySolveig - change - 16 Dec 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-12-16 15:33:45
Closed_By LadySolveig
avatar LadySolveig LadySolveig - close - 16 Dec 2023
avatar LadySolveig LadySolveig - merge - 16 Dec 2023
avatar LadySolveig
LadySolveig - comment - 16 Dec 2023

Thank you for your work @Fedik !
I am also very happy about the eager testers and the review - thank you @JonasAtZwetschke @viocassel and @dgrammatiko

Add a Comment

Login with GitHub to post a comment