-
Notifications
You must be signed in to change notification settings - Fork 12
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
Flyout: overlapping content #395
Comments
Hi @thephez! Thanks for the feedback. Currently, we only support enabling/disabling the floating flyout as you figured it out already. We are working together with theme authors to integrate this data into their own themes so it doesn't cover the useful content. |
Related to |
It also takes up content space on mobile |
As @thephez said,
I notice that the Just being able to configure the |
We are working on this approach. The goal is to leave theme authors the chance to define the web component on their theme like @ferdnyc @thephez would that be useful for your use case? We will really appreciate your feedback at this development stage. |
Yes, having the option in the dashboard would be helpful for me. Hopefully themes will eventually be able to integrate it well too. But the dashboard option shown in #482, plus the ability to customize the appearance, would definitely be helpful in the near term. 👍 |
We shipped the location in the Addons settings, so you should at least be able to update it now. Let us know if that works for you. |
Thanks! This gives us some flexibility. Looks like it worked as expected 👍 |
@ericholscher Being able to manually set the location in the project setting seems to work fine; I notice it doesn't require a rebuild of the docs (nice!), but does seem to require a force-reload of the page to get it to switch location, at least in Windows Chrome 131.0.6778.266 (Official Build) (64-bit). Still, that's minor. I'll need to experiment with the theme-driven location feature... I'm using the Furo theme which hasn't been updated to use Thanks! |
Hmm. Sorry, I guess I'm not clear from the discussion here or in #482 exactly where the I created a fork of Furo and added it to the footer code for the page, and while I see the tag in the source for my test project, the flyout hasn't moved. Is |
It's a body tag, yes. It doesn't have to be at any specific location when using |
@humitos Thanks, that's good to hear. Augh! And the problem in my test project is that I used |
I can confirm that when you actually use the correct tag name, it works as expected. 🫢 |
Is there a way to customize the location of the flyout? The switch from a static location on the bottom left to this floating thing on the right is pretty invasive for themes that take advantage of the right side of the screen. I didn't see anything on https://docs.readthedocs.io/en/stable/addons.html#enabling-read-the-docs-addons about how to adjust this and don't see an option on the dashboard other than enabling/disabling.
Front conversations
The text was updated successfully, but these errors were encountered: