You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When you navigate to the notification page after opening at least one post going back in history (using the browser back arrow for example) doesn't work. The url changes but the pages remains the notifiication's one.
I've noticed that this seems to be related with post's modals since if you don't open them before going on notification page this bug doesn't occurs.
Steps to Reproduce Issue
1. Open a post
2. Close the post's modal3. Open notification page4. Navigate back
Solution Proposed
I think the problem is that modals are put in navigation history (in order to close them navigating back), but they aren't properly removed from history when manually closed.
Environment
No response
Browsers
No response
OS
No response
Version of daily.dev
No response
Additional Context
No response
Code of Conduct
I follow the conditions of this project.
The text was updated successfully, but these errors were encountered:
Hey @miroirstudiosrl I wanted to work on this issue, but I am not able to reproduce this issue on my end here. Can you provide screenshots of the issue, if possible?
Hey @miroirstudiosrl I wanted to work on this issue, but I am not able to reproduce this issue on my end here. Can you provide screenshots of the issue, if possible?
@ParthTyagi15 was able to reproduce it. First go to any feed -> click any post -> modal will opened -> close the modal -> click the notifications page -> click browser back button. If you skip clicking any post, it should not reproduce. So the issue is definitely the way we handle the URL change on opening the modal. There should be a hook called usePostModalNavigation that facilitates the processing.
@sshanzel Thank you for the reply. Yes, now I am able to see the issue. It only happens if I close the post modal by clicking somewhere outside the Modal. If I close the post using the back arrow in the browser and then go to the notifications page(or any other page for that matter) and then again go back, the issue is not there. So I also agree it's the way we handle the URL change.
What went wrong? 🤔
When you navigate to the notification page after opening at least one post going back in history (using the browser back arrow for example) doesn't work. The url changes but the pages remains the notifiication's one.
I've noticed that this seems to be related with post's modals since if you don't open them before going on notification page this bug doesn't occurs.
Steps to Reproduce Issue
Solution Proposed
I think the problem is that modals are put in navigation history (in order to close them navigating back), but they aren't properly removed from history when manually closed.
Environment
No response
Browsers
No response
OS
No response
Version of daily.dev
No response
Additional Context
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: