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

feat(ui): change style about navbar #2421

Merged
merged 12 commits into from
Mar 24, 2018
Merged

feat(ui): change style about navbar #2421

merged 12 commits into from
Mar 24, 2018

Conversation

bnjjj
Copy link
Contributor

@bnjjj bnjjj commented Mar 20, 2018

TODO: change in package.json
Signed-off-by: Benjamin Coenen benjamin.coenen@corp.ovh.com

Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
ui/package.json Outdated
@@ -59,7 +59,7 @@
"ng2-codemirror-typescript": "^1.1.1",
"ng2-completer": "^1.4.0",
"ng2-dragula": "^1.2.1",
"ng2-semantic-ui": "^0.9.6",
"ng2-semantic-ui": "git://github.com/bnjjj/ng2-semantic-ui.git#feat_search_empty_query",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https ??

Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
@sguiheux
Copy link
Collaborator

remove space between navbar and list of project
remove scrollbar

@fsamin
Copy link
Member

fsamin commented Mar 22, 2018

in all projects list:

  • display project key
  • replace "Filter" by "Search"
  • ability to search by key

Signed-off-by: Benjamin Coenen <benjamin.coenen@hotmail.com>
Signed-off-by: Benjamin Coenen <benjamin.coenen@hotmail.com>
Signed-off-by: Benjamin Coenen <benjamin.coenen@hotmail.com>
@yesnault
Copy link
Member

when we load navbar, we should refresh recently viewed (to not display something renamed or deleted)

@bnjjj
Copy link
Contributor Author

bnjjj commented Mar 22, 2018

@yesnault will be done in another PR for resync with deletions

@bnjjj
Copy link
Contributor Author

bnjjj commented Mar 22, 2018

ready to merge

Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
Copy link
Member

@fsamin fsamin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix the build

Signed-off-by: Benjamin Coenen <benjamin.coenen@hotmail.com>
Signed-off-by: Benjamin Coenen <benjamin.coenen@hotmail.com>
@bnjjj
Copy link
Contributor Author

bnjjj commented Mar 24, 2018

fixed

@yesnault yesnault merged commit 5a9cd46 into master Mar 24, 2018
@yesnault yesnault removed the review label Mar 24, 2018
@bnjjj bnjjj deleted the style_navbar branch March 29, 2018 10:15
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

4 participants