Skip to content

Improve team members and repositories settings UI#5457

Merged
lafriks merged 5 commits into
go-gitea:masterfrom
lunny:lunny/improve_team_ui
Dec 9, 2018
Merged

Improve team members and repositories settings UI#5457
lafriks merged 5 commits into
go-gitea:masterfrom
lunny:lunny/improve_team_ui

Conversation

@lunny
Copy link
Copy Markdown
Member

@lunny lunny commented Dec 4, 2018

The team settings UI is something confusing and this PR will refactor that UI. It removed the links and add tab to switch team members and team repositories. It should fix #2691. See below screenshots.

jietu20181208-215902

@codecov-io
Copy link
Copy Markdown

codecov-io commented Dec 4, 2018

Codecov Report

Merging #5457 into master will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5457      +/-   ##
==========================================
+ Coverage   37.63%   37.64%   +<.01%     
==========================================
  Files         317      317              
  Lines       46824    46826       +2     
==========================================
+ Hits        17623    17626       +3     
  Misses      26699    26699              
+ Partials     2502     2501       -1
Impacted Files Coverage Δ
routers/org/teams.go 0% <0%> (ø) ⬆️
modules/process/manager.go 81.15% <0%> (+4.34%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da5a0b8...945b3ca. Read the comment docs.

@bkcsoft bkcsoft added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 4, 2018
@lafriks lafriks added type/enhancement An improvement of existing functionality topic/ui Change the appearance of the Gitea UI labels Dec 4, 2018
@lunny lunny force-pushed the lunny/improve_team_ui branch from 285bda3 to 9a8b9ee Compare December 8, 2018 13:48
@lunny lunny added this to the 1.7.0 milestone Dec 8, 2018
@lunny lunny changed the title WIP: improve team members and repositories settings UI Improve team members and repositories settings UI Dec 8, 2018
@lunny lunny mentioned this pull request Dec 8, 2018
6 tasks
@lunny
Copy link
Copy Markdown
Member Author

lunny commented Dec 8, 2018

It's ready to review. @go-gitea/maintainers

Comment thread options/locale/locale_en-US.ini Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should be No repositories can be accessed by this team.

Comment thread options/locale/locale_en-US.ini Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should be No members in this team.

@lunny lunny force-pushed the lunny/improve_team_ui branch from ca204c6 to 945b3ca Compare December 9, 2018 02:30
@lunny
Copy link
Copy Markdown
Member Author

lunny commented Dec 9, 2018

@lafriks done.

@techknowlogick
Copy link
Copy Markdown
Member

LGTM

@bkcsoft bkcsoft added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Dec 9, 2018
@bkcsoft bkcsoft 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 Dec 9, 2018
@lafriks lafriks merged commit 7c0c965 into go-gitea:master Dec 9, 2018
@lunny lunny deleted the lunny/improve_team_ui branch December 9, 2018 07:19
@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

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 type/enhancement An improvement of existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

consistency of the UX (in team view)

5 participants