search

migcontrol / django-migcontrol

A Wagtail-based Django website for Migration Control
GNU General Public License v3.0
0 stars 5 forks source link

Typo in Library navigation bar #248

Open teabag12 opened 6 months ago

teabag12 commented 6 months ago

under 'Library', remove ':' behind the word 'Library' in the navigation bar

benjaoming commented 6 months ago

Does it refer to this colon?

image

Strictly speaking, that's not a typo but a structural element to indicate that "Library" is the upper level and the other ones are nested inside that level.

Removing it, IMO, would require an alternative solution to visualize what's going on.

teabag12 commented 6 months ago

Yes, that's what it's referring to. Aah yes, that makes sense. Maybe we can change "Library:" to "All" or something like that?

benjaoming commented 6 months ago

I'm not sure that I understand how "All" communicates that the other items are nested sub-pages. They don't contain things that the library contains.

The issue might well be that "Library" itself is a list of media, and the other 2 items are different lists of data.

Maybe we need to give that list a name? And then have a landing page for the Library that explains what's going on in the 3 sub sections?


                            Library

           /                   |                 \
          /                    |                  \
Articles and books    Border Profiteers        Monthly Reviews
benjaoming commented 6 months ago

Since there seems to be some confusion, do you think you can discuss a bit with others? It'd be nice if this is intuitive for everyone.

teabag12 commented 6 months ago

Sure thing. I'll meet the others next week and come back to you on this :)