? ? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
11 Apr 2021

Pull Request for Issue #33067 .

Summary of Changes

After each sample data, there are progress bar/messages in list items and hidden by default.
The last bottom border is from the sample data and technically not the last child, thus it is not hidden.
Move the progress bar/messages into the sample data list item and not make them be list items.

Testing Instructions

Go to Home Dashboard
See no bottom border in Sample Data box.

avatar Quy Quy - open - 11 Apr 2021
avatar Quy Quy - change - 11 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Apr 2021
Category Modules Administration
avatar brianteeman
brianteeman - comment - 11 Apr 2021

Why not just apply the css that was proposed

avatar Quy
Quy - comment - 11 Apr 2021

The last visible border is not the last child.

33106

avatar brianteeman
brianteeman - comment - 11 Apr 2021

It works perfectly

.list-group-item:nth-last-child(3) {
    border-bottom: none;
}

image

image

avatar Quy
Quy - comment - 11 Apr 2021

If it can be done with less markup without resorting to CSS, then wouldn't that be better?

avatar brianteeman
brianteeman - comment - 11 Apr 2021

I am generally not a fan of nested lists

avatar Quy
Quy - comment - 11 Apr 2021

This PR moves the nested list so technically it is the same as before.

avatar ChristineWk ChristineWk - test_item - 11 Apr 2021 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 11 Apr 2021

I have tested this item successfully on 1440536


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

avatar brianteeman
brianteeman - comment - 11 Apr 2021

This PR moves the nested list so technically it is the same as before.

It's not but I can't be bothered on such a minor minor issue

avatar sandramay0905 sandramay0905 - test_item - 12 Apr 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 12 Apr 2021

I have tested this item successfully on 1440536


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

avatar Quy Quy - change - 12 Apr 2021
Status Pending Ready to Commit
Labels Added: ?
avatar Quy
Quy - comment - 12 Apr 2021

RTC


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

avatar rdeutz rdeutz - change - 14 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-14 08:17:39
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 14 Apr 2021
avatar rdeutz rdeutz - merge - 14 Apr 2021

Add a Comment

Login with GitHub to post a comment