search

nottinghamtec / PyRIGS

📅 A Django-based event booking system designed for use by TEC PA and Lighting, a 'Student Run Service' of the University of Nottingham Students Union
https://nottinghamtec.co.uk/
Other
25 stars 8 forks source link

Link to event details from check-in notice #595

Closed jb3 closed 1 month ago

jb3 commented 1 month ago

Again a feature requested earlier today.

This hyperlinks the check-in notice so you can quickly browse event details before checking in.

image

jb3 commented 1 month ago

I've changed the contrast a little:

image

It does make it not as easily recognizable as a link, but it does have hover feedback & the old version wasn't a link at all so it's still an improvement.

If people really care I can switch to a button but I think this is fine.

jerbzz commented 1 month ago

I like this feature, I'd go back to the default link colour though

jb3 commented 1 month ago

I like this feature, I'd go back to the default link colour though

Sure, I'll swap it back :-)

jb3 commented 1 month ago

Commit was reverted in the force push :+1:

FreneticScribbler commented 1 month ago

The default link colour on an 'info' coloured box is too low contrast - even the revised version was a little low (why is the default a shade of grey, not a proper black, these days?)

Looking at it again, I'm tempted to override bootstrap's CSS to use a white/black font on the alerts instead of one with the theme colour so it's actually vaguely readable - ala the other places the theme colours are used, in buttons etc. "alert-light" is a crime!

jb3 commented 1 month ago

Have pulled in the changes locally, one method to I guess ensure contrast on the blue background is to just use a very different colour, I don't hate it being red.

image