Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[2.0.14 Bug] Menu Item > Link Type > Link Icon class #346

Closed
joomleb opened this issue Aug 18, 2023 · 12 comments
Closed

[2.0.14 Bug] Menu Item > Link Type > Link Icon class #346

joomleb opened this issue Aug 18, 2023 · 12 comments

Comments

@joomleb
Copy link

joomleb commented Aug 18, 2023

Hi guys,
PHP 7.4.33 + Joomla 4.3.3 + Helix Ultimate 2.0.14
Menu Item home (for ex.) > Link Type > Link Icon class: fas fa-home (for ex.)

The font awesome Home icon is not shown in the Menu Item title (it is running perfectly, as expected, with the default Cassiopeia template)

Please, Can you address us on How to fix the bug ?
A "conflict" cannot be the explanation because I tested it on a clean Joomla 4 new installation

@joomleb
Copy link
Author

joomleb commented Sep 9, 2023

@ahamed @mi-prakash
...Please, Can you confirm and suggest a quick fixing ?

@mi-prakash
Copy link
Collaborator

@joomleb
Thanks for the feedback. I've found the issue
Hopefully it'll be fixed on next update
Best regards

@joomleb
Copy link
Author

joomleb commented Sep 14, 2023

@mi-prakash Thank you very much for your confirmation. I studied it and tried to solve the bug without success. Hooping to help you on solving, I leave here some useful informations:

1 - Has this Helix Ultimate bug been solved ?
joomla/joomla-cms#37871 (comment)

2 - Font Awesome 5 pseudo-elements fas "Solid" are free, but also fab "Brands" are free. To call "Font Awesome 5 Free" does not call Brand fab, to call them we have to call "Font Awesome 5 Brands".
So, both families should be called, something like:
font-family: 'Font Awesome 5 Free','Font Awedome 5 Brands';

3 - I did not understand if the "font-weight: 900" for fas Solid has been fixed in the last FA5 release (some icons in font awesome 5 not working without font-weight: 900;)

4 - Joomla 5 will introduce FA6 joomla/joomla-cms#36906 (comment). >. joomla/joomla-cms#40524
and Font Awesome icon packs are not backward compatible . Fas become fa-solid and fab fa-brands:
https://fontawesome.com/v6/docs/web/setup/upgrade/pseudo-elements#contentHeader

5 - Can be that the .../templates/shaper_helixultimate/css/template.css file:
li>a:before{display:none}
is generating the issue ?

6 - Can be that the Single-Colon is generating an issue ?

7 - Can be useful this Joomla 4 Forum discussion ?

@joomleb
Copy link
Author

joomleb commented Sep 15, 2023

@mi-prakash
Today I saw the Helix Ultimate 2.0.15 release, in the changelog:

  • Menu icon not working

So, I installed a new test Joomla 4.3.4 site and I installed last Helix Ultimate, you can see it running here

As far as I tested:

  • A - the Link Icon class (fas and fab) is running, thank you
  • B - into the Offcanvas the icon is not following the menu item color. In my opinion that should be the default behaviour, Do you agree?
  • C - I'm a "responsive, non-waster of space" supporter. Please, Can you set the margin-right in rem as ... ?:
    margin-right: .4rem;

D - Please, What is the Helix Ultimate plan for Joomla 5 /Font Awesome 6 ?

@joomleb
Copy link
Author

joomleb commented Oct 25, 2023

@mi-prakash
2.0.16 has been released.-As far as I see we can consider this as "solved", Right ?
If so, you can close this to keep GitHub clean.

(I cannot check because I'm traveling) Please, What is your decision on point "C" ?

@joomleb
Copy link
Author

joomleb commented Nov 19, 2023

@mi-prakash
Be careful, this bug is not solved for me, because you added a fixed "p-2" attribute class to the Link Icon class.
That is spacing also on the left side, moving all the menu items on the right and creating confusion with the sub menu items that are not using icons.

It should be "pe-2"
https://getbootstrap.com/docs/5.3/utilities/spacing/
Please, Can you confirm it ?
Please, Where can we manually edit and fix it ?

Maybe, it would be better to add it through CSS instead to use the fixed Bootstrap attribute. It would be simpler for HU customers to customize it, if needed.

Please, Why you choose to add there a padding space instead of a margin space ?

@mi-prakash
Copy link
Collaborator

mi-prakash commented Nov 22, 2023

@joomleb
sorry for misunderstanding
i've checked it, generally p-2 wont seems to have any issue but yes pe-2 will be a better solution here
we will try to improve this in the neext update using pe-2
in the meantime the issue remains closed. if in the next update u still have any issue, open another issue in git
thank you for understanding

@joomleb
Copy link
Author

joomleb commented Nov 22, 2023

@mi-prakash thank you for your reply.
Sure, I'll stay tuned.
Please, What is the file I can manually edit to change it to pe-2 ?
Can you address me?

@joomleb
Copy link
Author

joomleb commented Nov 29, 2023

Hi @mi-prakash
I found the same bug on standard Joomla Cassiopeia.
I discussed it here joomla/joomla-cms#42368. and I'm waiting the PR joomla/joomla-cms#42415 merging.

From my point of view "no predefined spacing attribute class" is better then "pe-2" class.
Sire, it cannot be "p-2".

Here the Helix Ultimate:overrides from Joomla

  • …/plugins/system/helixultimate/overrides/mod_menu/default_component.php - line 39 < …/modules/mod_menu/tmpl/default_component.php - line 44 and 47
  • …/plugins/system/helixultimate/overrides/mod_menu/… line 24 < …/modules/mod_menu/tmpl/default_heading.php - line 23 and 26
  • …/plugins/system/helixultimate/overrides/mod_menu/… line 24 < …/modules/mod_menu/tmpl/default_separator.php - line 23 and 26
  • …/plugins/system/helixultimate/overrides/mod_menu/… line 38 < …/modules/mod_menu/tmpl/default_url.php - line 36 and 39

Waiting for your confirmation, thanks...

@joomleb
Copy link
Author

joomleb commented Dec 1, 2023

@mi-prakash as far as I see into the 2.0.18 release, you choosen to add the pe-2 attribute class (instead of nothing).
Please, Can you confirm ?
(Just to know it to give us the opportunity to organize our works, thank you in advance)

@mi-prakash
Copy link
Collaborator

yes

@joomleb
Copy link
Author

joomleb commented Dec 29, 2023

Hi @mi-prakash
I'm back here because
PHP 8.1.26 + Joomla 4.4.1 + Helix Ultimate 2.0.18

The "B" issue: "...into the Offcanvas the icon is not following the menu item color. In my opinion that should be the default behaviour,.."
mentioned here #346 (comment). seems to be just partially solved because now:

f I add:
Menu Item > Link Type > Link class: blabla

And then into the Custom CSS
a.blabla span {
color: #ce1126;
}

The color is not applied to the menu icon into the Offcanvas position, while it is applied correctly outside the Offcanvas position.

It seems to be due for an abuse of the important attribute (strongly discouraged), a "similar" issue here #318

Please, Can you fix it ?
The Custom CSS should run without the need to add one time more the !important attribute and the ">" sibmenus icon should not be affected;
Screenshot_20231229-112054290~2

You can investigate it here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants