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

[5.2] Improve UX for template buttons - replace PR #43592 #43975

Merged
merged 3 commits into from
Aug 27, 2024

Conversation

chmst
Copy link
Contributor

@chmst chmst commented Aug 24, 2024

Pull Request for Issue #43212 .
Replaces #43592

Summary of Changes

Several users mentioned confusion i the com_templates toolbar, caused by the delete-file and close-file button.
This is an attempt to provide a solution with as few changes as possible.
Some buttons are used in the whole system and cannot be changed for this view, so we have not many options for changes here.

Changes made here:

Button for Preview always at position 1, becaus it is always vivile also for non-super usres.

Change the text on the "close file" button to "close". Because this button does not only close the file but it closes the screen and redirects back, for users it does not matter if there is another action in the code.

Move the "Close" Button" directly after the action buttons, before the file management buttons.

Use the trash icon on the "file delete" button. We had many discussions about that - In Joomla the trash button means "trashed" => state = -2 not "deleted permanently", but when we think as a user not as a developer, the icon is completely clear here.

I improved the code a little bit by removing an unused variable and used switch instead of nested elseif.

Testing Instructions

Play around in templates and template styles, make child templates, manage files.
Whatever you want to do with your template should have a comprehensive and complete toolbar, in most cases the same as before.

Actual result BEFORE applying this Pull Request

see #43212.
delete button and close-file button.
grafik

Expected result AFTER applying this Pull Request

grafik

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

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-5.2-dev labels Aug 24, 2024
@chmst chmst changed the title Improve UX for template buttons - replace PR #43592 [5.2] Improve UX for template buttons - replace PR #43592 Aug 24, 2024
@ChristineWk
Copy link

I have tested this item ✅ successfully on f45b391


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

@exlemor
Copy link

exlemor commented Aug 24, 2024

I have tested this item ✅ successfully on f45b391

I have tested this successfully. :)


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

@Quy
Copy link
Contributor

Quy commented Aug 24, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 24, 2024
@pe7er pe7er self-assigned this Aug 26, 2024
@pe7er pe7er enabled auto-merge (squash) August 26, 2024 14:05
@pe7er pe7er merged commit f2e0cd0 into joomla:5.2-dev Aug 27, 2024
3 of 4 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 27, 2024
@Quy Quy added this to the Joomla! 5.2.0 milestone Aug 27, 2024
@pe7er
Copy link
Contributor

pe7er commented Aug 27, 2024

Thanks @chmst !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators PR-5.2-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants