feat: Preserve last visited tab

[gopikrishnan13]

• Implemented functionality to store and retrieve the last visited tab index in localStorage.
• The application now defaults to the last visited tab after a page refresh, rather than always returning to the Activity tab.
• This ensures that users remain on the same tab they were on before refreshing the page, improving user experience.

Screencast.from.09-08-24.12.45.17.PM.IST.WEBM

Closes #192

[shariquerik]

@gopikrishnan13 This is not a correct way to fix this.
We should be able to change tabs via url
E.g /crm/leads/CRM-LEAD-2024-0001#activity_tab

And we can then store the last visited tab in local storage
So if url does not have #activity_tab in it then it will show last visited tab and if it has it then that tab will be loaded

[gopikrishnan13]

Hi @shariquerik

I made some updates based on your feedback and implemented the feature in the suggested way. it works fine.

I wrote this code in Lead.vue

Here’s the output video for your review.

Screencast.from.20-09-24.02.27.01.PM.IST.webm

Can I continue with this and make a commit?

Let me know if you have any further suggestions!

[shariquerik]

@gopikrishnan13 Yeah go ahead

[gopikrishnan13]

@shariquerik I updated this PR, please check it now,

• Finalized the implementation of useActiveTabManager for tab management.
• Enhanced logic to retrieve and set the active tab from both the URL and local storage.
• Set default behavior to the first tab if no valid active tab is found.
• Streamlined code for better readability and maintainability.

Screencast.from.25-09-24.06.00.23.PM.IST.webm

[shariquerik]

🎉 This PR is included in version 1.24.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[shariquerik]

@gopikrishnan13 There is one problem occurred after this.
When you are in Deal Page and when you go back using browsers back button it does not go back it just removes #activity hash from the url and then again when you go back it goes back
And when you go forward it goes to /deals/#activity and now you cannot go back it just lost history somehow

It is very irritating

Fixed: 2f58da9

[gopikrishnan13]

@shariquerik I have another solution, I’ve tested it, and it works well. Instead of setting the hash directly, I replace the history state only the first time when setting the tab from local storage. This way, it doesn’t affect the browser history and this prevents adding extra history entries

Do you think this is a better solution?