-
-
Notifications
You must be signed in to change notification settings - Fork 152
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
[WIP] Add storage interactions in the bottom-drawer #397
base: develop
Are you sure you want to change the base?
Conversation
@Pamplemousse have you got any sites you recommend testing with? |
https://jsfiddle.net/Benjol/HMEVd/ failed too :/ |
959ea83
to
d747ecd
Compare
This pull request introduces 1 alert when merging d747ecd into 1fdbb47 - view on LGTM.com new alerts:
Comment posted by LGTM.com |
@psiinon In both examples, the storage tab in the bottom-drawer is working, as long as the code interacting with the Storage API is run in the domain on which the HUD is effectively run. On https://www.w3schools.com/html/tryit.asp?filename=tryhtml5_webstorage_localThe demo code seems to be written in an iframe. On https://jsfiddle.net/Benjol/HMEVd/Investigating with the developer console, it appears that the localStorage concerned by the example is Just as a mention (not to be tackled) |
Will be needed to feed the bottom-drawer with storage or DOM events.
There will be other tabs, so try to make the current content generic. * css will not impact only history tab * there will be other messages passing, so rename current events related to history to be specificly for it
To know if the `localStorage` or `sessionStorage` is impacted.
storage_tool_field_action = Action | ||
storage_tool_field_key = Key | ||
storage_tool_field_time = Time | ||
tool_storage_field_type = Type |
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.
should be storage_tool_field_type
;)
Hey @Pamplemousse I must be doing something wrong. I'm still not seeing anything in storage: Do i need to do something to install the |
Ah, sorry I did not take the time to give you an update on this... Having seen the amazing and gigantic work that @jaywon has started in PR #459, I thought it would be smarter for me to wait for a cleaner and refactored structure to build my feature upon rather than hacking it into the current UI. This is still in a corner of my mind! |
Using the front-end-tracker to get storage interactions, then interface that to the HUD's bottom-drawer.
Here a couple of things to have a look at before integrating to the HUD:
getItem
is happening)history
tab