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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 BUG: Daily.Dev widget overlay covers elements on some sites #967

Open
1 task done
bobbyiliev opened this issue Aug 29, 2023 · 6 comments
Open
1 task done

Comments

@bobbyiliev
Copy link

What went wrong? 馃

The Daily.Dev widget overlaps with the elements on some websites.

For example, on this Hashicorp post here, there is a live-chat like widget that can not be closed as the Daily.Dev widget is exactly on top of the close button.

It works if you visit it on a larger screen, but on a 13" laptop it seems to be an issue as shown in the screenshot.

Expected Behavior

Maybe add a quick hide button to hide the widget temporarily just for this current page/session.

Steps to Reproduce Issue

Visist the Hashicorp blog from above using a smaller laptop screen, open the chat window, and you will be able to see the issue.

Solution Proposed

No response

Screenshots

![](https://github-production-user-asset-6210df.s3.amazonaws.com/21223421/263967628-6d904b43-29a4-48b8-af9a-733d7892dc78.png)

Environment

MacOS Chrome

Browsers

Chrome

OS

Mac

Version of daily.dev

3.26.3

Additional Context

No response

Code of Conduct

  • I follow the conditions of this project.
@idoshamun
Copy link
Member

Hi @bobbyiliev,

Thanks a lot for the report! 馃檹

@Powell-v2
Copy link

Hi @idoshamun 馃憢 I am eager to pick this one up as my first contribution to your awesome project. Just wanted to confirm first if it's still an issue?

@idoshamun
Copy link
Member

@Powell-v2 thanks a lot!! We still don't have designs for that. Let me check if we can squeeze that into the design backlog

@sshanzel
Copy link
Member

We came up with the idea of having the widget be movable, hopefully, with this change, we can solve the primary concern as the placement is going to be dynamic. The key change here would be making the top property to be based on the adjustments the user will make.

https://www.sketch.com/s/2549ccf9-b98c-413f-96fb-81b05b59e65f

@mrswapnilsahu
Copy link

Hi @sshanzel,

Can I work on this issue?

@sshanzel
Copy link
Member

Hi @mrswapnilsahu, thank you again for your interest, we're in the middle of fixing the extension for local development. Once done, I will ping you here with a comment 馃檶

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants