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

Limit utility navigation output, do not nest #61

Open
amckenz opened this issue Sep 11, 2018 · 3 comments · May be fixed by #284
Open

Limit utility navigation output, do not nest #61

amckenz opened this issue Sep 11, 2018 · 3 comments · May be fixed by #284
Labels
On Dev List 🔥 Product bug A bug which is not directly related to a custom child theme.

Comments

@amckenz
Copy link
Contributor

amckenz commented Sep 11, 2018

From @ashleykolodziej on May 15, 2018 20:23

Responsive Version: 2.0

Expected behavior:

Don't allow nested links and limit menu output to something reasonable, like the first three items.

Actual behavior:

screen shot 2018-05-15 at 4 18 38 pm

Link(s) where this can be reproduced:

http://sites-staging.bu.edu/ranzani-lab/

Related ServiceNow Ticket number (if available):

Apparently this ticket comes in about twice a month according to the #webteam folks

Copied from original issue: bu-ist/responsive-framework-1x#400

@ashleykolodziej ashleykolodziej added this to the Ready for Self Service milestone Jan 8, 2019
@ashleykolodziej ashleykolodziej added 🔥 Product bug A bug which is not directly related to a custom child theme. and removed 🎉 Enhancement labels Aug 20, 2021
@alana29s alana29s linked a pull request Oct 4, 2021 that will close this issue
7 tasks
@alana29s
Copy link
Contributor

@amckenz

I've started on this issue and have some additional questions.

  1. Can you please clarify the desired behavior. Is it only the top level items, and no levels of nesting, or only one level of nesting that shows in the screen shot with a limit of 3 items.
  2. Is three for the top level of the menu an official number that has been rolled out? Has 3 been enforced by IS&T?
  3. Do we need to come up with a plan to see if this will impact any existing sites?
  4. I have discovered methods to limit the output and have some initial work on HTC in my sandbox. I have limited the output to only the top level menu and 3 items below that. As stated in point 1 I believe this is not exactly what is requested. However, I can adapt the work I've completed already to the desired goal.
    HTC Menu Screen
    HTC Sandbox
  5. I've added a Warning Banner when the Utility menu is selected informing the user of the limits. Unfortunately, there are no hooks or filters to put this information in a different spot.
  6. Next on my list to investigate is seeing if I can remove any items beyond the limits when the "Save Menu" button is pressed and notify the users. Additionally, checking into the JavaScript drag and drop interface to see if I can prevent menu items from being placed when the limit is exceeded.

@ashleykolodziej
Copy link
Contributor

@stonybrookweb

  1. Only top level items and no nesting at all, please.
  2. There hasn't been an official number rolled out - this is a good catch. A limit through the code is probably premature and may cause confusion. I see you added some text to the UI on your sandbox - can we do something like "The utility menu does not support nested navigation items. More items may display in Menu Structure below, but these items will not display on your site. A maximum of four items in the utility menu is recommended."
  3. This behavior wasn't ever intended to be possible, and the (now live) site no longer has this navigation problem: https://sites.bu.edu/ranzani-lab/. I can give Ron a heads up so he knows how to respond to any tickets that come in, but I think it is safe to say that this is very unlikely to affect any live sites, and if it does, we can handle the tickets as they arise. Go ahead and fix in both 2x and 3x.
  4. This is awesome! Love the UI text. See 1 for specific requirements.
  5. I noticed the banner right away - I think this will work for our purposes.
  6. That would be phenomenal!

@alana29s
Copy link
Contributor

@ashleykolodziej
I'll update the depth to only allow top level menu items.
I will not limit the amount of items that can be output.
I'll adjust the message to be more specific to the desired outcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Dev List 🔥 Product bug A bug which is not directly related to a custom child theme.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants