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

V5 dashboard restyle #6214

Merged

Conversation

rashidkpc
Copy link
Contributor

Updated screenshot as of 4622a4c
screen shot 2016-02-11 at 5 00 35 pm

This adds the following to #6190:

  • Addressed @jbudz's comments
  • Added a dashed border on hover so you can tell where the borders of your panel are when moving and resizing
  • Changed buttons to match header text
  • Tightened up some padding

@rashidkpc rashidkpc mentioned this pull request Feb 11, 2016
@rashidkpc rashidkpc assigned jbudz and panda01 and unassigned jbudz Feb 11, 2016
@@ -4,10 +4,10 @@
**A:** Kibana 4 packages are architecture specific. Ensure you are using the correct package for your architecture.

**Q:** Where do I go for support?
**A:** Please join us at [discuss.elastic.co](discuss.elastic.co) with questions. Your problem might be a bug, but it might just be a misunderstanding, or feature we could improve. We're also available on Freenode in #kibana
**A:** Please join us at [discuss.elastic.co](https://discuss.elastic.co) with questions. Your problem might be a bug, but it might just be a misunderstanding, or feature we could improve. We're also available on Freenode in #kibana
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this comes from master. Minor, but there.

@simianhacker
Copy link
Member

LGTM

simianhacker added a commit that referenced this pull request Feb 12, 2016
@simianhacker simianhacker merged commit 2ce4dfb into elastic:feature/design Feb 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants