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

Fixed: Consistency of "unpublished" -i con colours #43375 #43424

Open
wants to merge 4 commits into
base: 4.4-dev
Choose a base branch
from

Conversation

mabdelaziz77
Copy link

Fixed the inconsistency of the unpublished icon colour of the articles in the associations modal window.

Pull Request for Issue #43375 .

Summary of Changes

Changed the unpublished icon from icon-times (red) to icon-unpublish (grey) in the article associations modal window.

Testing Instructions

  1. Make sure the multi-language feature is enabled by enabling the System - Language Filter plugin.
  2. Install a second language if you have only one language installed.
  3. Create an article for each language, set their state to unpublished.
  4. Edit one of the two articles, go to the Associations tab and click the Select button to select an article.
  5. Make sure the unpublished icon is grey.

Actual result BEFORE applying this Pull Request

The unpublished icon color is red.
unpublished icon is red

Expected result AFTER applying this Pull Request

The unpublished icon is grey.
unpublished icon is grey

Fixed the icon colour of the unpublished articles in the associated articles list.
@Quy
Copy link
Contributor

Quy commented May 3, 2024

There are more instances. Please find the following and apply the same change. Thank you!

0 => 'icon-times',

@mabdelaziz77
Copy link
Author

There are more instances. Please find the following and apply the same change. Thank you!

0 => 'icon-times',

Replaced the other instances.

@brianteeman
Copy link
Contributor

not that simple

image

you also need to change icon-folder to icon-archive

@Quy
Copy link
Contributor

Quy commented May 4, 2024

Also, please change icon-check to icon-publish.

@mabdelaziz77
Copy link
Author

I noticed that all the modal windows have the same issue. Is this look intentional?
I'm not sure if it’s supposed to be like this, so is there any reference we can check to see if this is the expected behavior?
Let me know!

@chmst
Copy link
Contributor

chmst commented May 4, 2024

@mabdelaziz77 thank you very much for your work!
Differences are not intentional, modal views should look same as the normal views. Probably testers forgot testing modal views in recent versions.

@Quy
Copy link
Contributor

Quy commented May 10, 2024

I have tested this item ✅ successfully on 7364302


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants