Skip to content
This repository was archived by the owner on Apr 17, 2023. It is now read-only.

Ui minor changes and responsiveness of the website improved. #477

Merged
mssola merged 16 commits intomasterfrom
ui-minor-changes
Oct 22, 2015
Merged

Ui minor changes and responsiveness of the website improved. #477
mssola merged 16 commits intomasterfrom
ui-minor-changes

Conversation

@cyntss
Copy link
Copy Markdown
Contributor

@cyntss cyntss commented Oct 22, 2015

Plus: on android (iphone not tested but should have a similar behaviour), adding the website to the home will give an effect of having installed a native app removing the URL bar on top and also has an app icon.

@mssola
Copy link
Copy Markdown
Collaborator

mssola commented Oct 22, 2015

LGTM 👍 Some notes:

  • Maybe the placeholder of the search element should say "Search repository" (instead of "repo").
  • The size of the icon on notifications is incosistent. See the uploaded images.

logos1
logos2

@mssola
Copy link
Copy Markdown
Collaborator

mssola commented Oct 22, 2015

Don't worry about the test failures, I'll fix them after we merge this ;)

@cyntss
Copy link
Copy Markdown
Contributor Author

cyntss commented Oct 22, 2015

@mssola fixed ;)

mssola added a commit that referenced this pull request Oct 22, 2015
Ui minor changes and responsiveness of the website improved.
@mssola mssola merged commit 415900a into master Oct 22, 2015
@mssola mssola deleted the ui-minor-changes branch November 12, 2015 10:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants