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.

Sign up for GitHub

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

Jump to bottom

gui: Various improvments #3257

Closed
wants to merge 1 commit into from
Closed

gui: Various improvments #3257

wants to merge 1 commit into from

Conversation

norgeous
Copy link
Contributor

@norgeous norgeous commented Jun 2, 2016

before:
untitled

after:
untitled

@AudriusButkevicius
Copy link
Member

Feels like a childrens game, find X differences, where X is undefined.
So far my calculations say that X is 1.

@norgeous
Copy link
Contributor Author

norgeous commented Jun 2, 2016

sorry my bad. to elaborate...

  1. fixed logo placement under 768 width. now aligned to 'left margin' and vertically centred in the header bar.
  2. gap between icon and title fixed - as i believe i have broken this previously
  3. added .panel-icon class to all accordion and modals - this allows for those h3s and h4s to be spread out onto multiline (which i believe is prefered)

@norgeous
Copy link
Contributor Author

norgeous commented Jun 2, 2016

clarification:

  • css file changes: points 1 and 2 above
  • html file changes: all the html files i have changed, i have just added <div class="panel-icon"> and multilined

@AudriusButkevicius
Copy link
Member

@st-review merge

gui: CSS tweaks for mobile views

@st-review
Copy link

👌 Merged as 3795a78. Thanks, @norgeous!

@st-review st-review closed this Jun 2, 2016
st-review pushed a commit that referenced this pull request Jun 2, 2016
@norgeous @AudriusButkevicius
gui: CSS tweaks for mobile views
3795a78 
GitHub-Pull-Request: #3257
@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label Jun 16, 2017
@syncthing syncthing locked and limited conversation to collaborators Jun 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@norgeous @AudriusButkevicius @st-review