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

Rework repo buttons (#20602, #20718) #20719

Merged
merged 7 commits into from
Aug 20, 2022

Conversation

silverwind
Copy link
Member

@silverwind silverwind commented Aug 8, 2022

Backport #20602 and #20718 to 1.17. Translations can be backported separately.

* Rework repo buttons

- Replace "New PR" and "Go to File" button with Icon Button
- Move all "Add File" actions into a dropdown button
- Remove most custom styling of clone buttons
- Margin and wiki tweaks

Buttons are now all equal height, mobile layout wraps gracefully.

Fixes: go-gitea#13671
Replaces: go-gitea#20375

Co-authored-by: Lauris BH <lauris@nix.lv>
Co-authored-by: zeripath <art27@cantab.net>
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
@silverwind silverwind added this to the 1.17.1 milestone Aug 8, 2022
@Gusted Gusted added the topic/ui Change the appearance of the Gitea UI label Aug 9, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Aug 9, 2022
…ea#20718)

Fix two regressions from go-gitea#20602:

- Restore the 'History' button that was previously unable to render
  because it's show condition was never hit
- Hide the 'Add File' button when there would be no items in the
  dropdown.

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@silverwind silverwind changed the title Rework repo buttons (#20602) Rework repo buttons (#20602, #20718) Aug 9, 2022
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Aug 17, 2022
@zeripath
Copy link
Contributor

Translations can be backported separately.

I think that is a recipe for forgetting to backport them at all. We should do this now assuming that a good number of translations have editor.add_file set in them in main

@zeripath zeripath modified the milestones: 1.17.1, 1.17.2 Aug 17, 2022
@zeripath
Copy link
Contributor

Pushed to 1.17.2 - not enough translations ready.

@silverwind
Copy link
Member Author

Yeah, will check for translations

@silverwind
Copy link
Member Author

@jolheiser was so nice to backport the translations. We can land this if we land #20834 immediately afterwards.

@zeripath
Copy link
Contributor

make lgtm work

@zeripath zeripath merged commit c61ed6f into go-gitea:release/v1.17 Aug 20, 2022
@silverwind silverwind deleted the backport-20602 branch August 20, 2022 19:35
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants