Opened 3 months ago

Last modified 3 months ago

#30936 assigned defect

clean up pronoun display in people page

Reported by: catalyst Owned by:
Priority: Medium Milestone:
Component: Webpages/Website Version:
Severity: Normal Keywords:
Cc: traumschule, trac-dip-importer Actual Points:
Parent ID: Points:
Reviewer: hiro Sponsor:

Description

The people page would look better if we hid parts of the pronoun.is URL in the visible link text, e.g., displaying pronoun.is/they instead of https://pronoun.is/they. The link would still go to the same place, though. (This also matches the conventions of a few other pages I've seen that use pronoun.is URLs.)

Merge request in https://dip.torproject.org/web/tpo/merge_requests/1

Child Tickets

Change History (5)

comment:1 Changed 3 months ago by teor

I also wonder why the link is in capitals, which are harder to read.

(And why there isn't any space between IRC: and nick, but that's probably a separate ticket.)

comment:2 in reply to:  1 Changed 3 months ago by catalyst

Replying to teor:

I also wonder why the link is in capitals, which are harder to read.

That seems to come from h6 { /*...*/ text-transform: uppercase; /*...*/ } in one of the CSS files, but it's buried in some SCSS stuff that I don't understand. I'm not sure about the reasoning behind that design choice either.

(And why there isn't any space between IRC: and nick, but that's probably a separate ticket.)

That's much easier to answer: it's missing in templates/macros/people.html.

comment:3 Changed 3 months ago by emmapeel

i will remove the capitalization as part of #30027

comment:4 Changed 3 months ago by catalyst

Status: newneeds_review

Oops I should have set this to needs_review

comment:5 Changed 3 months ago by gaba

Owner: hiro deleted
Reviewer: hiro
Status: needs_reviewassigned
Note: See TracTickets for help on using tickets.