Superfish Submenu Indicator Arrow Won't Display When js/css Aggregation Turned On

Created on 22 March 2022, about 3 years ago
Updated 28 April 2023, about 2 years ago

Problem/Motivation

The down or right arrows that indicate submenus are not displayed when js/css aggregation is enabled. When the js/css aggregation is turned off the arrows show up in the Superfish menus.

Steps to reproduce

Add a submenu to a menu. Turn js/css caching on.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

πŸ› Bug report
Status

Needs review

Version

1.4

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States donpwinston

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡΅πŸ‡±Poland saramato

    I have exactly the same problem. Only on the taxonomy terms pages. With js aggregation enabled.
    A message appears in the console:
    Uncaught ReferenceError: assignment to undeclared variable p
    The solution described in #7 does not help.

  • πŸ‡ΊπŸ‡ΈUnited States donpwinston

    #6 doesn't really work. As soon as you flush the cache again the down arrows disappear again. Why doesn't some fix this? I'm using Drupal 10 and the same problem occurs.

  • Issue was unassigned.
  • Status changed to Active almost 2 years ago
  • Status changed to Postponed: needs info about 1 month ago
  • πŸ‡§πŸ‡ͺBelgium lobsterr

    I have tried to reproduce this issue different ways, but unfortunately I can't. They only thing I noticed, we are missing arrows just a sec and then when CSS is applied they correctly displayed and once it is cached then they appear immediately.

    Could you provide more information how to reproduce it?
    What version is used and etc. ?

  • πŸ‡§πŸ‡ͺBelgium lobsterr

    I think, I found the problem and sometimes it was broken for me too. The cause of the problem was in special symbols used directly in CSS:

    content: "β–Ί";

    When aggregation happens, these symbols are not correct encoded. So, the proper way to use UTF codes instead:

    content: "\25BA";

    I will add fix in the next version of the library.

    Thank you for your contribution

  • πŸ‡§πŸ‡ͺBelgium lobsterr
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024