add support section to sidebar in admin ui

[aaaaargZombies]

fixes #2146

Added all the suggested links into the sidebar and moved the report a bug link in there too, I wrote the AC as appearing for admins but I included roots in this too as there seemed no harm in it and being familiar with what other users will see will make it easier to advice / support people.

[kimadactyl]

I think this looks great, my only q is how do people feel about mailto: links? I feel like for me they are a bit of very old fashioned tech that often results in my mail client opening and trying to run me through a setup wizard. Maybe it should just be a plain text email address? Prob ideally it would take you to a web form inside PlaceCal that fills in extra details for us automatically actually, but one for another day.

I think if we keep the mailto link at least lets make the text the email address rather than "email us" so u don't have to mouse over to find the info?

[aaaaargZombies]

I think the problem of not using mailto: in a link in a link tag is an email isn't a valid url and it breaks like this

we could just leave it as text?

[kimadactyl]

yeah i was advocating for removing the link completely and just tell people the email. fine if others disagree ofc! ive just litearlly never found this function to be welcome personally.

[katjam]

Agree with kim. Plan text is perfect and more usable.

[aaaaargZombies]

I see! That makes sense, it's implemented now

[kimadactyl]

this still has the "email us" link in - need to remove?

also looks a bit out of place in the menu - would it make sense to move it into the small text below the build number?

[aaaaargZombies]

whoops took the screenshot too soon

you can see it's removed in the code here

    <h6 class="sidebar-heading d-flex justify-content-between align-items-center px-3 mt-4 mb-1 text-muted">
      <span>Get support</span>
    </h6>
    <%= admin_nav_link('Handbook', 'https://handbook.placecal.org/', 'book')%>
    <li class="nav-item"><span class="nav-link"><i class="fa fa-envelope feather"></i>[email protected]</span></li>
    <%= admin_nav_link('Discord', 'http://discord.gfsc.studio/', 'comment')%>
    <%= admin_nav_link('Report a bug', 'https://github.com/geeksforsocialchange/PlaceCal/issues/new?assignees=&labels=&template=bug_report.md', 'bug')%>

view code in context

[aaaaargZombies]

[aaaaargZombies]

also looks a bit out of place in the menu - would it make sense to move it into the small text below the build number?

I really think we should have all the get support stuff in one place not scattered around the shop - I felt "email us" looked more in keeping but as you and Katja say maybe less practical.

[kimadactyl]

ok i see what you did now didnt take it in properly, agreed looks good. could maybe just put it at the top or bottom of the list but otherwise all cool!

[aaaaargZombies]

It's in the bottom of the list for non-root users, the bottom of the list is all that stuff that should probably be ignored because we aren't working on it, collections etc.

[kimadactyl]

sorry i specifically mean put the email link at the top or bottom of the list - itll prob look a bit more balanced to go "ABBB" than "ABAA"

[aaaaargZombies]

OK I think we might be on the home straight