joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

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

Open mabdelaziz77 opened this issue 1 year ago • 11 comments

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

mabdelaziz77 avatar May 03 '24 17:05 mabdelaziz77

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

0 => 'icon-times',

Quy avatar May 03 '24 19:05 Quy

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

0 => 'icon-times',

Replaced the other instances.

mabdelaziz77 avatar May 03 '24 20:05 mabdelaziz77

not that simple

image

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

brianteeman avatar May 03 '24 23:05 brianteeman

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

Quy avatar May 04 '24 00:05 Quy

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!

mabdelaziz77 avatar May 04 '24 10:05 mabdelaziz77

@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.

chmst avatar May 04 '24 17:05 chmst

I have tested this item :white_check_mark: successfully on 73643025a84c179a346df5becb596087b8a3eccb


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

Quy avatar May 10 '24 15:05 Quy

I have tested this item :white_check_mark: successfully on 7cb128d0afdd183622cc1ee6eec765262c5e1024


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

chmst avatar Aug 24 '24 11:08 chmst

I have tested this item :white_check_mark: successfully on 7cb128d0afdd183622cc1ee6eec765262c5e1024


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

mabdelaziz77 avatar Aug 24 '24 13:08 mabdelaziz77

RTC


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

Quy avatar Aug 24 '24 13:08 Quy

I have tested this item :white_check_mark: successfully on 7cb128d0afdd183622cc1ee6eec765262c5e1024

I tested successfully both for articles and categories

gacompa avatar Aug 24 '24 14:08 gacompa

Thanks!

laoneo avatar Aug 30 '24 05:08 laoneo