NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
31 May 2021

Adds a more visible highlight for the active option

this is an scss change so dont forget to rebuild

pr for #34305
before

avatar brianteeman brianteeman - open - 31 May 2021
avatar brianteeman brianteeman - change - 31 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 May 2021
Category Administration Templates (admin) NPM Change
avatar Krshivam25 Krshivam25 - test_item - 31 May 2021 - Tested successfully
avatar Krshivam25
Krshivam25 - comment - 31 May 2021

I have tested this item successfully on b62ed89


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

avatar Quy
Quy - comment - 31 May 2021

Heading is not readable.

34310

avatar Krshivam25
Krshivam25 - comment - 31 May 2021

@Quy Please test it again
Screenshot (254)

avatar brianteeman brianteeman - change - 31 May 2021
Labels Added: NPM Resource Changed ?
avatar joomla-cms-bot joomla-cms-bot - change - 31 May 2021
Category Administration Templates (admin) NPM Change Administration com_config Templates (admin) NPM Change
avatar brianteeman
brianteeman - comment - 31 May 2021

@Quy good spot - please retest

avatar Krshivam25
Krshivam25 - comment - 31 May 2021

@brianteeman Sorry I forget to type npm run build:css in terminal to rebuild the files.

avatar Quy
Quy - comment - 31 May 2021

Can you make the entire row clickable and not just text?

avatar brianteeman
brianteeman - comment - 31 May 2021

any suggestions

avatar Quy
Quy - comment - 31 May 2021

Add display: block; here should do the trick.

    a {
      font-weight: $font-weight-normal;
      color: var(--atum-text-dark);
      text-decoration: none;

Edit: This is an issue before the PR: Also there is flickering between cursor and hand when moving up and down through the items.

a9a5efa 31 May 2021 avatar brianteeman block
avatar brianteeman
brianteeman - comment - 31 May 2021

added the block - i can see the cursor thing but cant see the fix

avatar Quy
Quy - comment - 31 May 2021

Sorry I don't know either. Maybe @ciar4n can help fix it in a separate PR?

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

I have tested this item successfully on a9a5efa


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

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

I have tested this item successfully on a9a5efa


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

avatar PhilETaylor
PhilETaylor - comment - 31 May 2021

If you want to fix all the comments, and also make it look better, then use this:

// Sidebar

.sidebar-nav {
  padding: 1rem 0 1rem 2rem;

  li {
    font-size: .9rem;
    font-weight: $font-weight-bold;

    &.divider {
      margin: .3rem 0;
    }

    a {
      padding: .25rem;
      display: block;
      font-weight: $font-weight-normal;
      color: var(--atum-text-dark);
      text-decoration: none;

      &::before {
        margin-inline-end: .5rem;
        font-family: "Font Awesome 5 Free";
        font-weight: 900;
        content: "\f054";

        [dir="rtl"] & {
          content: "\f053";
        }
      }
    }

    &.item:hover, &.active {
      background-color: var(--atum-bg-dark-60);

      a {
        color: var(--atum-text-light);
      }
    }
  }
}
avatar PhilETaylor
PhilETaylor - comment - 31 May 2021
avatar Quy Quy - test_item - 31 May 2021 - Tested successfully
avatar Quy
Quy - comment - 31 May 2021

I have tested this item successfully on 672fbc7

Perfect!


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

avatar sandramay0905 sandramay0905 - test_item - 1 Jun 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 1 Jun 2021

I have tested this item successfully on 672fbc7

Thanks all for creating, comment and test the pull request.


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

avatar richard67 richard67 - change - 1 Jun 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 1 Jun 2021

RTC


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

avatar chmst chmst - change - 6 Jun 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-06-06 21:08:31
Closed_By chmst
Labels Added: ?
avatar chmst chmst - close - 6 Jun 2021
avatar chmst chmst - merge - 6 Jun 2021
avatar chmst
chmst - comment - 6 Jun 2021

Thanks!

avatar brianteeman
brianteeman - comment - 6 Jun 2021

great thanks

Add a Comment

Login with GitHub to post a comment