Opened 3 years ago

Closed 3 years ago

#21380 closed enhancement (fixed)

Onionoo web-page(s) should have new Metrics design

Reported by: iwakeh Owned by: metrics-team
Priority: Medium Milestone: Onionoo-1.2.0
Component: Metrics/Onionoo Version:
Severity: Normal Keywords:
Cc: RaBe Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

web-page should have new Metrics design

what else?

Child Tickets

Change History (24)

comment:1 Changed 3 years ago by karsten

Some ideas:

  • The list of Onionoo clients can go away. (The introductory paragraph before that should probably stay.)
  • We could easily merge the remainder of the home page with the protocol page into a single page.
  • Protocol versions should be a section of its own.
  • Example usage should be moved closer to the top before we dive too much into technical details.
  • We might need a box with links to anchors on the page. We don't have that on Tor Metrics, because we have many shorter pages there. But here we have one long page (or two, if we can't agree on the single-page change), so we'll need at least some navigation.
  • We don't have a logo for Onionoo. We could just use the normal Tor logo.

comment:2 Changed 3 years ago by iwakeh

Sounds fine. The one-page change is a good one.
(Maybe, we keep the navigation discussion on #21381?)

As with CollecTor: Should we include a link to the wiki mirror list or have mirrors listed here?

comment:3 in reply to:  2 Changed 3 years ago by karsten

Replying to iwakeh:

Sounds fine. The one-page change is a good one.
(Maybe, we keep the navigation discussion on #21381?)

You mean the discussion how to implement the navigation box with links or similar? Yes, happy to.

As with CollecTor: Should we include a link to the wiki mirror list or have mirrors listed here?

Yes, we can do that.

comment:4 Changed 3 years ago by RaBe

Here's the Onionoo page - let me know what you think :) onionoo.cc-ltd.net

comment:5 Changed 3 years ago by karsten

Thanks for starting this! A few early comments:

  • The two sections "Developing Onionoo applications" and "Related projects" should go away.
  • We can probably remove the "Protocol overview" section header, because the entire page exists only to give an overview of the Onionoo protocol.
  • We should change all <h3> to <h1>, all <h4> to <h2>, etc.
  • Navigation topic again: should the table of contents box be visible in the mobile view, too?
  • The list under "Methods" looks broken in the mobile view.
  • Maybe we'll need better colors than the currently used blue and red for indicating new and deprecated protocol parts. Is it easy to find colors that play well with the purple and green?

comment:6 in reply to:  5 Changed 3 years ago by RaBe

Replying to karsten:

  • The two sections "Developing Onionoo applications" and "Related projects" should go away.

Removed.

  • We can probably remove the "Protocol overview" section header, because the entire page exists only to give an overview of the Onionoo protocol.

Changed the headline.

  • We should change all <h3> to <h1>, all <h4> to <h2>, etc.

Fixed.

  • Navigation topic again: should the table of contents box be visible in the mobile view, too?

Added a mobile navigation.

  • The list under "Methods" looks broken in the mobile view.

Fixed.

  • Maybe we'll need better colors than the currently used blue and red for indicating new and deprecated protocol parts. Is it easy to find colors that play well with the purple and green?

Made the red and blue less red and blue :)

comment:7 Changed 3 years ago by karsten

I like it! If iwakeh has no concerns, I'll apply that design later today or tomorrow morning and we can further tweak it from there.

comment:8 Changed 3 years ago by iwakeh

Status: newmerge_ready

I like it, too!

Just some minor things:

  • It's odd that the button "tell me more" simply scrolls down a bit. We could just do without the grey top-panel and add that text before the "General" headline. That way the navigation would be visible immediately when accessing the site.
  • And, another color topic: already with the old page I thought it counter-intuitive to have green for "required"; I think it could be simple black as optional is grey. That would give a bigger color range for the protocol colors, i.e., orange could be the new red and green the new blue?

But, these are just minor tweaks. All fine.

comment:9 Changed 3 years ago by iwakeh

Just notice that the title says "CollecTor" not Onionoo.

comment:10 Changed 3 years ago by RaBe

I fixed the title, and I made "required" black. I'd rather not make the new entries green, because that is our link color. For the grey jumbotron box: I like it for optical reasons - and we use that same box on CollecTor and MetricsLib (that you all haven't seen yet) :)

comment:11 Changed 3 years ago by RaBe

I added "new", "beta" and "deprecated" labels, what do you think?

comment:12 in reply to:  11 Changed 3 years ago by iwakeh

Replying to RaBe:

I added "new", "beta" and "deprecated" labels, what do you think?

That's a nice idea!

comment:13 Changed 3 years ago by karsten

Good ideas above! I'll now apply changes. Stay tuned, this shouldn't take long. An hour maybe?

comment:14 Changed 3 years ago by iwakeh

Text change: The onionoo-announce list mentioned under 'versions' doesn't exist anymore. This can just be removed as there is the next_major_version_scheduled field now.

comment:15 Changed 3 years ago by karsten

There, pushed to my redesign branch and deployed on the server. Thoughts?

comment:16 Changed 3 years ago by karsten

Ah, RaBe, can you rescue the design mess I left on the protocol page? :)

comment:17 in reply to:  15 Changed 3 years ago by iwakeh

Replying to karsten:

There, pushed to my redesign branch and deployed on the server. Thoughts?

Fine, only that 'required' looks still green to me.

comment:18 Changed 3 years ago by RaBe

Great!

Could you please redownload the css file of the prototype page, there were slight color corrections (required, and the label colors matching the texts).

For the protocol page, just remove the class "page-header" from the box with the content.

comment:19 in reply to:  18 Changed 3 years ago by karsten

Replying to RaBe:

Great!

Could you please redownload the css file of the prototype page, there were slight color corrections (required, and the label colors matching the texts).

Done!

For the protocol page, just remove the class "page-header" from the box with the content.

Done!

Anything else that's broken and needs fixing?

comment:20 Changed 3 years ago by karsten

Status: merge_readyneeds_review

Should we merge what we have and release it together with protocol version 4.0 next Tuesday?

comment:21 Changed 3 years ago by iwakeh

Milestone: Onionoo-1.2.0

Yes, added the milestone.

comment:22 Changed 3 years ago by karsten

So, merge_ready? :)

comment:23 Changed 3 years ago by iwakeh

Status: needs_reviewmerge_ready

Oh, I thought I did that already; hmm, wondering what else received the merge_ready ;-)

comment:24 Changed 3 years ago by karsten

Resolution: fixed
Status: merge_readyclosed

You indeed set it to merge_ready before, but we made more changes afterwards, so I set it back to needs_review earlier today. Anyway, merged and pushed to master. Thanks, RaBe and iwakeh! Closing.

Note: See TracTickets for help on using tickets.