Skip to content
This repository was archived by the owner on Oct 17, 2025. It is now read-only.

Conversation

@Chew
Copy link
Member

@Chew Chew commented Jan 1, 2021

image

Mainly here for feedback and improvements.

@rtm516
Copy link
Member

rtm516 commented Jan 1, 2021

Don't think this needs to use all the navbar classes, and we should put the content in a container? Just so it's the same width as the rest of the page

@Chew Chew added the frontend This issue or pull request relates to an issue/change on the frontend label Jan 7, 2021
@Chew
Copy link
Member Author

Chew commented Apr 3, 2021

Should the footer show up on auth? It does right now. If so, should the navbar too?

@rtm516
Copy link
Member

rtm516 commented Apr 8, 2021

Should the footer show up on auth? It does right now. If so, should the navbar too?

I would say no, auth pages look clean with no footer or header

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

Labels

frontend This issue or pull request relates to an issue/change on the frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants