-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Fix:(isolation mode issue) issue #1492 #1509
Conversation
Codecov Report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix!
changes regarding the isolated page view url...
Hello @sapegin,
|
🎉 This PR is included in version 10.4.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
issue link: #1492
This issue occurred because of the fix that have been made in version 10.0.1 for the issue: #1454
Hashed Url example: the "#button" in the url is an example of Hashed Url...it gets appended after clicking the Button component in the side-Navbar...
![Screenshot 2020-01-03 at 6 14 45 PM](https://user-images.githubusercontent.com/21274438/71724136-2967fc80-2e55-11ea-9a6e-39c9d16d21e2.png)
After the issue #1454 was resolved...it was taking the hashed url token in the isolated page(which opens after clicking the fullscreen icon) url as you can see below
after clicking the fullscreen icon we got results like this!
![pageNotFound(1)](https://user-images.githubusercontent.com/21274438/71724438-54068500-2e56-11ea-80c8-736a9591db60.png)
so this was the issue which is resolved by doing a simple check i.e. we are verifying if the currentHash that we are getting in the 'buildIsolatedOrNoChromeFragment()' function is a hashedUrl("#button") or a Section-Url("#/Components")...
we just checked the first character after the '#' in the currentHash string if it matches the hashedUrlpattern then we are neglecting the extra currentHashPath...
now as you can see below we removed the "/button" from the url...it is working fine..
![pageFound](https://user-images.githubusercontent.com/21274438/71725180-050e1f00-2e59-11ea-921c-e471a6be3df0.png)
we have written a test case for that as you can check...!
and also the issue mentioned in #1454 is also working fine...