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

People seem to have a hard time realizing the tabs are tabs #6

Open
grahamc opened this issue Jan 27, 2018 · 3 comments
Open

People seem to have a hard time realizing the tabs are tabs #6

grahamc opened this issue Jan 27, 2018 · 3 comments

Comments

@grahamc
Copy link
Member

grahamc commented Jan 27, 2018

A few times now I've shown people the viewer and they get stuck on the default screen showing the log about subscribing. They don't seem to notice the UUIDs show up, or realize that they should click on them.

@samueldr
Copy link
Member

Definitely!

I'm even surprised this comment wasn't the first thing you said to me, but I did describe them as tabs seeding the thought before you even had the chance to discover the UI.

I'll be looking into ways to present them as tabs, but I'm also interested in trying to keep this all looking like a TUI, any ideas in how it could be otherwise presented?

@samueldr
Copy link
Member

As shared on IRC

[23:16:26] <gchristensen> [...], I don't have great  ideas w.r.t. #6 but I tried something like this: https://screenshots.firefox.com/PjcF4dpi5d5vYM49/nix.samueldr.com
[23:17:02] <gchristensen> only use the first two segments of the uuid in the UI, a 14px light gray border between each "tab" 

screenshot-2018-1-27 log viewer nixos nixpkgs 34196

@grahamc
Copy link
Member Author

grahamc commented Jan 27, 2018

So I think:

  1. make the header background lightgray (for example) https://github.com/samueldr/logger-wip/blob/master/src/styles/index.less#L53
  2. make individual tabs have a declared background color of @color_header_bg
  3. give individual tabs a right-margin, ex:
.app > header ul > li {
    display: inline-block;
    background: #AFFFFF;
    margin-right: 1rem;
}

I think it'd be cool to shorten up the displayed UUIDs, using something like text-overflow: ellipsis so that the header can easily accommodate three builds on a typical screen size. I tried to do this, but ran out of know-how on CSS, in particular it did funny things when I hovered.

This is roughly what I tried:

.app > header ul > li {
    display: inline-block;
    background: #AFFFFF;
    margin-right: 1rem;
    max-width: 10rem;
    overflow: hidden;
}

span {
    text-overflow: ellipsis;
    overflow: hidden;
}

but I probably missed some stuff.

samueldr added a commit that referenced this issue Jan 27, 2018
samueldr added a commit that referenced this issue Jan 27, 2018
Part of #6, explains what this tool is.
samueldr added a commit that referenced this issue Jan 27, 2018
This may make the interface easier to understand.

For #6
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

No branches or pull requests

2 participants