? Pending

User tests: Successful: Unsuccessful:

avatar N6REJ
N6REJ
27 Jul 2020

Pull Request for Issue #30191

Summary of Changes

change icon class for "icon-save" to "fas fa-save"

Testing Instructions

go to administrator/index.php?option=com_users&view=group&layout=edit&id=9
via inspector verify both save button icons are using "fas fa-save" instead of "icon-save"

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Documentation Changes Required

none

avatar N6REJ N6REJ - open - 27 Jul 2020
avatar N6REJ N6REJ - change - 27 Jul 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Jul 2020
Category Layout
avatar infograf768 infograf768 - test_item - 28 Jul 2020 - Tested successfully
avatar infograf768
infograf768 - comment - 28 Jul 2020

I have tested this item successfully on 225e913


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

avatar sandewt sandewt - test_item - 28 Jul 2020 - Tested successfully
avatar sandewt
sandewt - comment - 28 Jul 2020

I have tested this item successfully on 225e913

I noticed that there are two spaces between "button-save btn btn-success".


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

avatar sandewt
sandewt - comment - 28 Jul 2020

I noticed that there are two spaces between "button-save btn btn-success".

#30204-test

avatar N6REJ
N6REJ - comment - 28 Jul 2020

I noticed that there are two spaces between "button-save btn btn-success".

#30204-test

There's another pr pending dealing with that whole button class issue.. I'll resolve it there.

avatar infograf768 infograf768 - change - 28 Jul 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-07-28 14:30:54
Closed_By infograf768
Labels Added: ?
avatar infograf768 infograf768 - close - 28 Jul 2020
avatar infograf768 infograf768 - merge - 28 Jul 2020
avatar infograf768
infograf768 - comment - 28 Jul 2020

tks

avatar sandewt
sandewt - comment - 28 Jul 2020

There's another pr pending dealing with that whole button class issue.. I'll resolve it there.

I see, #30206 ?

Add a Comment

Login with GitHub to post a comment