search

readthedocs / addons

JavaScript client to integrate with Read the Docs nicely
https://readthedocs-addons.readthedocs.io/
MIT License
21 stars 4 forks source link

Flyout: make it obvious there are features inside it #364

Open humitos opened 2 months ago

humitos commented 2 months ago

The logo takes up a lot of space and doesn't exactly communicate "click here for hosted features".

(@agjohnson in https://github.com/readthedocs/addons/pull/360#issuecomment-2286908369)

I agree that the logo would benefit from some change. The current state looks more like an advertisement for RTD than a feature of the opened project’s docs; this disturbs users' focus.

(@orangesunny in https://github.com/readthedocs/addons/pull/360#issuecomment-2288462774)

I'm opening this issue here to talk more about this and find a way to keep the Read the Docs branding on the flyout while communicating there are hosting features behind a click on the it.

There is an issue opened about re-designing the flyout completely as well at #53

agjohnson commented 1 month ago

It's also worth mentioning that we lost the dropdown caret in this version of the flyout. Without this visual clue, it's not clear the menu even expands:

image

I think the collapsed menu is not the best place to advertise RTD -- at very least for our full wordmark and logo combo. The user hasn't interacted with any features yet, which makes the full logo and wordmark on the flyout feel more like subtle brand placement than a feature users should interact with.

I think a solution is to move our full branding into the flyout, where the reader is interacting with our features. This leaves the most valuable real estate, the collapsed menu, open for features the reader/maintainers need. We're limited on how big we can make the flyout before it will become a nuisance.