Skip to content

Conversation

crowlKats
Copy link
Collaborator

@crowlKats crowlKats commented Jul 2, 2025

Closes #893

image

Screenshot 2025-07-02 at 14-47-05 Account Settings - JSR
Screenshot 2025-07-02 at 14-46-39 Account Settings - JSR

@AugustinMauroy
Copy link
Contributor

in AccountLayout you can add gitlab link

@crowlKats
Copy link
Collaborator Author

@AugustinMauroy i tried, however gitlab requires an auth token to get the username so we can resolve the link, whereas github doesnt. i added the logic there, but its commented out until we figure out a solution

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where this file is used ?

Copy link
Collaborator Author

@crowlKats crowlKats Jul 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh seems deno fmt changes formatting of a bunch of svgs. this is used explicitly in the docs gen for alerts

@lucacasonato
Copy link
Member

This sign in page needs an overhaul

Sign In with GitHub
</a>
<a
href="/login/gitlab"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not use tb icon like on setting page ? because now the gitlab icon on yellow bg is awful

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because from all other services that provide oauth2 logins, i see the actual logos being used, for easier brand recognition.
ie imagine adding a microsoft oath2 login. the logo in black is unrecognizable

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe change the background color because on this image it didn't look good

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Not Require Github Account

3 participants