Skip to content

Conversation

CirnoT
Copy link
Contributor

@CirnoT CirnoT commented May 23, 2020

This PR makes styling of tabular menu in forms consistent across site for arc-green and in-line with how it looks on default theme.

Before:
chrome_2020-05-23_02-39-46
chrome_2020-05-23_03-40-56

After:
chrome_2020-05-23_16-30-15
chrome_2020-05-23_16-30-04

The "After" styling was partially taken from what was previously applied only to Wiki form.

@silverwind
Copy link
Member

Maybe make the tab background one shade darker? It stands out a bit too much imho.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 23, 2020
@CirnoT
Copy link
Contributor Author

CirnoT commented May 23, 2020

Changed active item to be darker and updated screenshots.

CirnoT added a commit to CirnoT/gitea that referenced this pull request May 23, 2020
@GiteaBot GiteaBot 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 May 24, 2020
@zeripath zeripath added the topic/ui Change the appearance of the Gitea UI label May 25, 2020
@zeripath zeripath added this to the 1.13.0 milestone May 25, 2020
techknowlogick added a commit that referenced this pull request May 25, 2020
* Style code review comment for arc-green

* change border around review for light; update border around textarea to match #11570

Co-authored-by: zeripath <[email protected]>
Co-authored-by: techknowlogick <[email protected]>
@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 May 29, 2020
@zeripath
Copy link
Contributor

Make lg-tm work

@zeripath zeripath merged commit 4395c60 into go-gitea:master May 29, 2020
@CirnoT CirnoT deleted the arc-green_tabular branch May 29, 2020 23:34
@6543
Copy link
Member

6543 commented Jun 7, 2020

Pleace Backport

CirnoT added a commit to CirnoT/gitea that referenced this pull request Jun 7, 2020
* Make tabular menu styling consistent for arc-green

* rework completely

* transparent borders

* use darker color for active item; override only colors for borders

* Update web_src/less/themes/theme-arc-green.less

Co-authored-by: silverwind <[email protected]>

Co-authored-by: silverwind <[email protected]>
Co-authored-by: zeripath <[email protected]>
(cherry picked from commit 4395c60)
@lafriks lafriks added the backport/done All backports for this PR have been created label Jun 7, 2020
techknowlogick pushed a commit that referenced this pull request Jun 7, 2020
* Make tabular menu styling consistent for arc-green

* rework completely

* transparent borders

* use darker color for active item; override only colors for borders

* Update web_src/less/themes/theme-arc-green.less

Co-authored-by: silverwind <[email protected]>

Co-authored-by: silverwind <[email protected]>
Co-authored-by: zeripath <[email protected]>
(cherry picked from commit 4395c60)
ydelafollye pushed a commit to ydelafollye/gitea that referenced this pull request Jul 31, 2020
* Style code review comment for arc-green

* change border around review for light; update border around textarea to match go-gitea#11570

Co-authored-by: zeripath <[email protected]>
Co-authored-by: techknowlogick <[email protected]>
ydelafollye pushed a commit to ydelafollye/gitea that referenced this pull request Jul 31, 2020
* Make tabular menu styling consistent for arc-green

* rework completely

* transparent borders

* use darker color for active item; override only colors for borders

* Update web_src/less/themes/theme-arc-green.less

Co-authored-by: silverwind <[email protected]>

Co-authored-by: silverwind <[email protected]>
Co-authored-by: zeripath <[email protected]>
@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
backport/done All backports for this PR have been created 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