Add studiesocial support button, fixes #381

cthit / chalmersit-rails

Chalmers.it built with Rails

MIT License

4 stars 6 forks source link

Add studiesocial support button, fixes #381 #393

Closed NeonNeon closed 5 years ago

NeonNeon commented 5 years ago

@Baloo1 did u have a specific link in mind?

NeonNeon commented 5 years ago

Det är ju något som ser lite off ut, jag gissar att den nya knappen blir större eftersom den innehåller längre text, vilket resulterar i att det ser ut att vara lite mer whitespace mellan den och slack-knappen än de övriga?

NeonNeon commented 5 years ago

Om man kortar ner texten som synns på knappen så ser det bättre ut

Hoidi commented 5 years ago

Kan det stå studiehjälp eller nåt liknande? Tycker att det borde vara på svenska om man har det valt iallafall. Och bara "hjälp" kanske är lite för vagt. Men som du sa så ser det lite wack ut med den långa texten.

Vad tycker resten av @cthit/digit

NeonNeon commented 5 years ago

Studiehälsa då kanske? Lika många bokstäver och med Studiehjälp då tänker jag på läxhjälp

molleer commented 5 years ago

Studiehjälp det ut som ett bra alternativ

Hoidi commented 5 years ago

Studiehälsa låter jättebra

NeonNeon commented 5 years ago

Skulle vilja att @Baloo1 säger om länken är rimlig, eller om han vill ha något annat innan merge

lindskogen commented 5 years ago

@NeonNeon how does it look on smaller screens?

Baloo1 commented 5 years ago

Studiehälsa låter skitbra. Den länken är också bra, tack 😄

NeonNeon commented 5 years ago

Innan merge bör det testas på mindre skärm. Typ kolla mobil varianten av hemsidan. Har inte gjort det än

NeonNeon commented 5 years ago

Om jag öppnar i firefox -> meny -> web developer -> Responsive design så ser det ut såhär (vilket jag förstod som var mobil-vy.

Och det ser ju bra ut så jag tycker man kan mergea. Screenshot from 2019-10-22 22-11-43