I have tested this item ✅ successfully on 730815b

I have tested this item ✅ successfully on 730815b

rtc Thanks for testing

RTC

I tested that. I don't like important. I don't think we need that here, do we?

For me the star looked OK with

  .home-image {
    padding: .8rem 0;
  }

@astridx
!important is needed. Otherwise this takes over

with

.main-nav span.fas {
    padding: .9rem 0;
}

without !important we do not get the star aligned correctly

with important, we get

In fact what misses here is a small change for quickstart + icon

.main-nav li .menu-dashboard > a > .fas, .main-nav li .menu-quicktask > a > .fas {
    display: flex;
    align-items: center;
    padding: .9rem;    /////instead of 1rem
    margin: 0;
}

can do here or in another pr.

Thanks for explaining. I actually only looked at the second (horizontal) value . But I would also use no important for overwriting the the first one (vertical). I would do this:

home-image {
   padding: 0;
   &.fa-star {
   	padding: .8rem 0;
   }
 }

That only applies to the star, or am I missing something

Has hound stopped working again?

I have tested this item ✅ successfully on 730815b

Tested with:
Chrome 80.0.3987.122
PHP 7.3.14

@astridx
Corrected as suggested + also corrected aligment for dashboard and plus icons

Folks, please test again.

I tested this again.

1. git fetch origin pull/28131/head:4.0_batch_menuitems_star
2. git checkout 4.0_batch_menuitems_star
3. npm ci
   Everything is fine with the star. Thank you @infograf768 . I see no difference with the plus icon, it so little. But the code is uses.

Before and after

--

--

I have tested this item ✅ successfully on 2b3f8eb

I have tested this item ✅ successfully on 2b3f8eb

I have tested this item ✅ successfully on 2b3f8eb

Thanks