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

Fix layout #690

Merged
merged 6 commits into from
Jul 9, 2017
Merged

Fix layout #690

merged 6 commits into from
Jul 9, 2017

Conversation

kazup01
Copy link
Member

@kazup01 kazup01 commented Jul 7, 2017

Before:
screen shot 0029-07-09 at 12 49 29 pm

After:
screen shot 0029-07-09 at 12 50 16 pm

@kazup01 kazup01 requested a review from asmsuechan July 7, 2017 17:20
@asmsuechan
Copy link
Contributor

asmsuechan commented Jul 8, 2017

Personally, it might be better to put the text (Search) inside the input tag.

@kazup01
Copy link
Member Author

kazup01 commented Jul 8, 2017

screen shot 0029-07-08 at 3 58 19 pm

@kazup01
Copy link
Member Author

kazup01 commented Jul 8, 2017

Fix star position at NoteList

screen shot 0029-07-08 at 4 05 26 pm

@asmsuechan
Copy link
Contributor

It overlaps the tag area.
screen shot 2017-07-08 at 16 38 28

@kazup01
Copy link
Member Author

kazup01 commented Jul 9, 2017

Fixed layout in Tag area at NoteList

screen shot 0029-07-09 at 12 46 23 pm

@kazup01 kazup01 merged commit 7a4234e into master Jul 9, 2017
@kazup01 kazup01 deleted the update-layout branch July 9, 2017 04:29
@kohei-takata kohei-takata mentioned this pull request Jul 29, 2017
@kazup01 kazup01 mentioned this pull request Jul 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants