Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.0] Swap layout of buttons in com_media's delete confirmation modal #33221

Merged
merged 1 commit into from Apr 22, 2021

Conversation

YatharthVyas
Copy link
Contributor

Pull Request for Issue #33215 .

Summary of Changes

The delete button to the right of the cancel button in com_media's delete confirmation modal.
The reasoning behind this is better explained in #33215 (comment)

Testing Instructions

  1. npm ci
  2. Joomla Administrator Panel -> Sidebar -> Content -> Media
  3. Select any file and click on the delete button

Ensure that the delete button is to the right of cancel and it works as intended.

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Documentation Changes Required

None

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Apr 21, 2021
@YatharthVyas YatharthVyas changed the title [4.0] places delete button to the right of cancel [4.0] Swap layout of buttons in com_media's delete confirmation modal Apr 21, 2021
@ChristineWk
Copy link

I have tested this item ✅ successfully on 04f806f

With Prebuild Package


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

@ChristineWk
Copy link

With Patch:

screen shot 2021-04-21 at 21 32 20


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

@ghost
Copy link

ghost commented Apr 22, 2021

I have tested this item ✅ successfully on 04f806f


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

@joomla-cms-bot joomla-cms-bot removed NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Apr 22, 2021
@alikon
Copy link
Contributor

alikon commented Apr 22, 2021

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 22, 2021
@alikon
Copy link
Contributor

alikon commented Apr 22, 2021

RTC


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

@joomla-cms-bot joomla-cms-bot added RTC This Pull Request is Ready To Commit and removed RTC This Pull Request is Ready To Commit labels Apr 22, 2021
@chmst chmst added this to the Joomla 4.0 milestone Apr 22, 2021
@chmst chmst merged commit 90972b8 into joomla:4.0-dev Apr 22, 2021
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev and removed RTC This Pull Request is Ready To Commit labels Apr 22, 2021
@chmst
Copy link
Contributor

chmst commented Apr 22, 2021

Thanks!

@YatharthVyas YatharthVyas deleted the media-modal-btn-order-fix branch May 8, 2021 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NPM Resource Changed This Pull Request can't be tested by Patchtester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants