? NPM Resource Changed PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
25 Nov 2022

It always bugs me that the icon and text in the alert are off-center. (and someone else because there was an issue in the past but I cant find it now).

This simple PR sets the correct line height which re-centers the icon which I assume was the original intent based on the original css and the bootstrap alert it is inspired by.

To test you will need to either use a prebuilt package or apply the pr and npm run build:css

Before

image

After

image

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed
avatar brianteeman brianteeman - open - 25 Nov 2022
avatar brianteeman brianteeman - change - 25 Nov 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Nov 2022
Category Repository NPM Change
avatar brianteeman brianteeman - change - 4 Dec 2022
Labels Added: NPM Resource Changed PR-4.3-dev
avatar obuisard obuisard - change - 15 Jul 2023
Title
[4,3] joomla alert icons
[4.3] joomla alert icons
avatar obuisard obuisard - edited - 15 Jul 2023
avatar rachelwalraven rachelwalraven - test_item - 26 Aug 2023 - Tested successfully
avatar rachelwalraven
rachelwalraven - comment - 26 Aug 2023

I have tested this item ✅ successfully on 8489df2

On Joomla 4.4.dev with php 8.1.22


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

avatar RickR2H RickR2H - test_item - 26 Aug 2023 - Tested successfully
avatar RickR2H
RickR2H - comment - 26 Aug 2023

I have tested this item ✅ successfully on 8489df2


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

avatar RickR2H RickR2H - change - 26 Aug 2023
Status Pending Ready to Commit
avatar RickR2H
RickR2H - comment - 26 Aug 2023

RTC


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

avatar obuisard obuisard - change - 29 Aug 2023
Labels Added: ?
avatar obuisard obuisard - change - 29 Aug 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-08-29 02:17:51
Closed_By obuisard
avatar obuisard obuisard - close - 29 Aug 2023
avatar obuisard obuisard - merge - 29 Aug 2023
avatar obuisard
obuisard - comment - 29 Aug 2023

Thank you Brian @brianteeman !

Add a Comment

Login with GitHub to post a comment