Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Home button should be hidden when LANDING_PAGE is set to "explore" or "organizations" #4648

Closed
2 of 8 tasks
0rzech opened this issue Aug 8, 2018 · 2 comments · Fixed by #4651
Closed
2 of 8 tasks
Labels
topic/ui Change the appearance of the Gitea UI
Milestone

Comments

@0rzech
Copy link
Contributor

0rzech commented Aug 8, 2018

  • Gitea version (or commit ref): 1.4.3
  • Git version: Not relevant
  • Operating system: Not relevant
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
    • Not relevant
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist: Not relevant

Description

When app.ini contains LANDING_PAGE = explore, all of the three buttons (the cup, Home and Explore) point to the same place. This is redundant and kind of confusing. See here for an example.

So what would I expect? When LANDING_PAGE is set to home, both home and explore buttons should be visible, as they point to different locations. When LANDING_PAGE is set to explore or organizations, the home button should be hidden and explore button should just be labelled home, as it is it's real role now.

@lafriks lafriks added the topic/ui Change the appearance of the Gitea UI label Aug 8, 2018
@0rzech 0rzech changed the title "Explore" button should be hidden when LANDING_PAGE is set to "explore" "Explore" button should just be labelled "Home" when LANDING_PAGE is set to "explore" Aug 9, 2018
@0rzech 0rzech changed the title "Explore" button should just be labelled "Home" when LANDING_PAGE is set to "explore" "Explore" button should just be labelled "Home" when LANDING_PAGE is set to "explore" or "organizations" Aug 9, 2018
@techknowlogick
Copy link
Member

Perhaps instead of renaming explore to home, just remove the home link when the above situation occurs.

@0rzech
Copy link
Contributor Author

0rzech commented Aug 9, 2018

@techknowlogick The home button is removed in above situation. Explore is just additionally renamed to home, because this is exactly what explore button means when LANDING_PAGE is explore or organizations. Also, this will be consistent for people visiting different instances of Gitea. Home-labelled button will always be there for them, just defaulting to different pages. Of course, if you still want me not to rename explore in addition to hiding home, just let me know and I will update PR.

@0rzech 0rzech changed the title "Explore" button should just be labelled "Home" when LANDING_PAGE is set to "explore" or "organizations" "Home" button should be hidden when LANDING_PAGE is set to "explore" or "organizations" Aug 10, 2018
@0rzech 0rzech changed the title "Home" button should be hidden when LANDING_PAGE is set to "explore" or "organizations" Home button should be hidden when LANDING_PAGE is set to "explore" or "organizations" Aug 10, 2018
@lunny lunny added this to the 1.6.0 milestone Aug 27, 2018
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants