NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar himanshu007-creator
himanshu007-creator
27 May 2021

Pull Request for Issue #34174.

Summary of Changes

As discussed here
Changed the warning accent color to yellow and text color to mud yellow.

Testing Instructions

  • build css npm run build:css
  • check the warning alert ( you can try to login to backend with wrong credetials for example)
  • notice the colors of warning alert

Actual result BEFORE applying this Pull Request

A mud(ish) yellow color accent color (#996900)

Expected result AFTER applying this Pull Request

Yellow color in general and text color + dismiss button to mud(ish) yellow (#996900)
ss

Documentation Changes Required

none

avatar himanshu007-creator himanshu007-creator - open - 27 May 2021
avatar himanshu007-creator himanshu007-creator - change - 27 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 May 2021
Category Administration Templates (admin) NPM Change
avatar dgrammatiko
dgrammatiko - comment - 27 May 2021

The !important is unacceptable

avatar himanshu007-creator himanshu007-creator - change - 27 May 2021
Labels Added: NPM Resource Changed ?
avatar himanshu007-creator
himanshu007-creator - comment - 27 May 2021

The !important is unacceptable

Made the changes

avatar sandramay0905 sandramay0905 - test_item - 28 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 28 May 2021

I have tested this item successfully on 070905b

With Pull Request

Screenshot_2021-05-28 Joomla Update - sql - Administration(1)

Without Pull Request

Screenshot_2021-05-28 Joomla Update - sql - Administration(1)


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34246.
avatar YatharthVyas YatharthVyas - test_item - 28 May 2021 - Tested successfully
avatar YatharthVyas
YatharthVyas - comment - 28 May 2021

I have tested this item successfully on 070905b


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

avatar richard67 richard67 - change - 28 May 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 28 May 2021

RTC


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

avatar Quy Quy - change - 28 May 2021
Status Ready to Commit Pending
avatar himanshu007-creator himanshu007-creator - change - 29 May 2021
Labels Added: ?
avatar sandramay0905
sandramay0905 - comment - 30 May 2021

Is the pull request ready for a re-test?

avatar chmst
chmst - comment - 30 May 2021

The dismiss button must have the darker colour too.

avatar himanshu007-creator
himanshu007-creator - comment - 30 May 2021

The dismiss button must have the darker colour too.

will do it, Thanks

avatar himanshu007-creator himanshu007-creator - change - 30 May 2021
Labels Removed: ?
avatar himanshu007-creator
himanshu007-creator - comment - 30 May 2021

@sandramay0905 , @YatharthVyas please test this PR again.
Thanks

avatar himanshu007-creator himanshu007-creator - change - 30 May 2021
The description was changed
avatar himanshu007-creator himanshu007-creator - edited - 30 May 2021
avatar Krshivam25 Krshivam25 - test_item - 31 May 2021 - Tested successfully
avatar Krshivam25
Krshivam25 - comment - 31 May 2021

I have tested this item successfully on a8914d0


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

avatar sandramay0905 sandramay0905 - test_item - 31 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 31 May 2021

I have tested this item successfully on a8914d0

image


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34246.
avatar richard67 richard67 - change - 31 May 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 31 May 2021

RTC


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

avatar rdeutz rdeutz - change - 31 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-31 08:09:00
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 31 May 2021
avatar rdeutz rdeutz - merge - 31 May 2021
avatar PhilETaylor
PhilETaylor - comment - 1 Jun 2021

This only changed the color for the alert rendered by custom elements - the standard bootstrap alert alert-warning now is a glaring in-your-face yellow - see #34322

avatar PhilETaylor
PhilETaylor - comment - 1 Jun 2021

Please test #34328

Add a Comment

Login with GitHub to post a comment