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

Smaller UI and UX changes #186

Merged
merged 5 commits into from May 29, 2017
Merged

Smaller UI and UX changes #186

merged 5 commits into from May 29, 2017

Conversation

@lennartkoopmann
Copy link
Member

@lennartkoopmann lennartkoopmann commented May 26, 2017

Like in other parts of the web interface, I changed the way the buttons are used to navigate in the top part of the pages. The way they were changing from page to page made following the current position in the navigation really hard and jumping back almost impossible. We are now always using the same button composition and the current page or parent page is highlighted as active.

pipelinesnav

I added a prominent warning in case a pipeline is not connected to a stream. I noticed quite a few users who got stuck at this part of setting up pipelines because it is not immediately clear that you have to connect a pipeline to a stream and the current small warning is hard to notice.

screen shot 2017-05-26 at 6 28 49 pm

The navigation that was changing on pages was immensely confusing me. Like already done for other parts of the web interface that are following a similar pattern, this change addresses this by always having the same navigation items and highlighting the currently active one.
This helps a little with the large amount of buttons on the page.
Show a prominent warning on the pipeline details page in case the pipeline is not connected to any streams.
Include a note that you do not have to connect a pipeline to a stream if you intend to use it for decorators only. Also make clear that pipelines that are connected to a stream will process incoming messages.
Copy link
Member

@kroepke kroepke left a comment

lgtm

@kroepke kroepke merged commit 8e755b7 into master May 29, 2017
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
graylog-project/pr Jenkins build graylog-project-pr-snapshot 152 has succeeded
Details
@kroepke kroepke deleted the ux-ui-changes branch May 29, 2017
@kroepke kroepke added this to the 2.3.0 milestone May 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants