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

Double borders on edit file view #1152

Closed
2 tasks done
pgaskin opened this issue Mar 9, 2017 · 5 comments
Closed
2 tasks done

Double borders on edit file view #1152

pgaskin opened this issue Mar 9, 2017 · 5 comments
Labels
topic/ui Change the appearance of the Gitea UI
Milestone

Comments

@pgaskin
Copy link
Contributor

pgaskin commented Mar 9, 2017

  • Gitea version (or commit ref):
  • Git version:
  • Operating system:
  • Database (use [x]):
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (Just edit any file)
  • Log gist: Not relevant

Description

There is unnecessary double borders around the editors on the edit file view. This applies to both the markdown and the plain text editor.

This CSS code will fix it:

.repository.file.editor .tab[data-tab="write"] {
    padding: 0 !important;
}
.repository.file.editor .tab[data-tab="write"] .editor-toolbar {
    border: none !important;
}
.repository.file.editor .tab[data-tab="write"] .CodeMirror {
    border-left: none;
    border-right: none;
    border-bottom: none;
}

Screenshots

Before modifications After modifications
screenshot screenshot 1
@lunny lunny added the topic/ui Change the appearance of the Gitea UI label Mar 9, 2017
@lunny lunny added this to the 1.x.x milestone Mar 9, 2017
@lunny
Copy link
Member

lunny commented Mar 9, 2017

Why change the style? How the new one is better than old one?

@pgaskin
Copy link
Contributor Author

pgaskin commented Mar 9, 2017

The new one is more clean, and saves space. It is also more consistent with GitHub and the rest of the Gitea UI.

@pgaskin
Copy link
Contributor Author

pgaskin commented Mar 9, 2017

I have submitted PR #1153

@lunny lunny modified the milestones: 1.2.0, 1.x.x Mar 9, 2017
@pgaskin
Copy link
Contributor Author

pgaskin commented Mar 9, 2017

@lunny, could this go into 1.1.0, or is that one finished for now?

@lunny
Copy link
Member

lunny commented Mar 9, 2017

I think this should not be taken to v1.1, or we will never address 1.1. 1.1 has only 1 bug left.

lunny pushed a commit that referenced this issue Mar 10, 2017
* Fix double borders on edit page (#1152)

* Add border modifications to less file
@pgaskin pgaskin closed this as completed Mar 10, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 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

No branches or pull requests

2 participants