[4.0] com_menu - Change Featured Icon to Home Icon in

[coolcat-creations]

Pull Request for Issue #28976

Summary of Changes

Changed the Default Icon in Menus to a home icon. If it has to be called home or default could be discussed in another issue. This PR makes a difference between the featured - action and the set menu item as default action. It also adds the possibility to pass the active and inactive iconclass suffix to the isDefault method.

Testing Instructions

apply patch
see if the home Icon on monolingual sites is now a home icon
the non-default items are for now a simple circle
on a multilingual website the home icon should be still a flag like before.

Expected result

Sidebar:

Actual result

[infograf768]

Remark 1:

Size of icon is very different between sidemenu and Menu dasboard. Menu dashboard is better imho.

Remark 2:
Density of Yellow is very different between the featured star (for articles manager) and Home icon set to All languages in menu items manager. Articles manager star density is better imho.

The same colour is applied

color: var(--warning);
border-color: var(--warning);

The issue comes from

.disabled {
    opacity: .4;
}

These "issues" are unrelated to this patch but it would be good to solve them at the same time.

[coolcat-creations]

Yes, this PR is only about to change the type of the icon, not the color or size - for color we would need another class in the table header and for sizes it’s also not bound to only the home icon but the general interface. see later comment :-)

[adj9]

I have tested this item ✅ successfully on 109a7b2

Apply the PR and I've change the icon for menu.
I've do an article in person lang and see the icon should

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29016.}

[coolcat-creations]

I have tested this item ✅ successfully on 109a7b2

Apply the PR and I've change the icon for menu.
I've do an article in person lang and see the icon should

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

Thank you!

[coolcat-creations]

@adj9 @infograf768 I added the styling for the home item to not have the disabled class and added a "not-allowed" cursor. Can you retest?
For the icon sizes I think a separate PR would be better.

[infograf768]

will do tomorrow.

[infograf768]

I have tested this item ✅ successfully on 38bc020

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29016.}

[infograf768]

@coolcat-creations

I suggest you also add a few lines above

  &.featured-disabled {
    opacity: 1;
  }

as this will now also solve density in Installed Languages and Template Styles for the default star colour.

[N6REJ]

@coolcat-creations

  &.disabled .icon-home {
    color: $state-warning-bg;
    border-color: $state-warning-bg;
  }

should be

&.disabled .fas.fa-home,
  &.disabled .icon-home {
    color: $state-warning-bg;
    border-color: $state-warning-bg;
  }

While below is preferred, I understand for b/c reasons that may not be possible at this time.

  &.disabled .fas.fa-home {
    color: $state-warning-bg;
    border-color: $state-warning-bg;
  }

[coolcat-creations]

@N6REJ - I can't add code that is not applying yet, you would need to add this into your PR after mine is merged or me after yours is merged, but not before both are merged.

[coolcat-creations]

&.featured-disabled {
opacity: 1;
}

@infograf768 - May I create a seperate PR for that? I prefer not to mix up too much in case something needs to be looked up later or reverted.

[infograf768]

@infograf768 - May I create a seperate PR for that? I prefer not to mix up too much in case something needs to be looked up later or reverted.

Sure, but don’t forget! 😺

[infograf768]

I have tested this item ✅ successfully on 0838c6d

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29016.}

[Quy]

I have tested this item ✅ successfully on 0838c6d

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29016.}

[Quy]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29016.}

[infograf768]

Tks

[infograf768]

@coolcat-creations
Now you can make the PR for #29016 (comment) ;)