The background script should grab the user preferences from chrome storage first before setting to DEFAULTS 👍

Currently, the background script is setting defaults on install but I think this means it sets defaults when the browser updates too.

Nice extension so far, was testing it out and thought it was causing some redraw bugs across sites, but it was just my computer.

In the process of testing that I uninstalled the extension. Without reinstalling, I visited Twitter again and the focused aspect is still there. I'm assuming this is related to how Twitter caches things to make the PWA available offline.

I tried deleting my cookies/cache and it didn't fix the problem. I'm not sure if this is something that can be included in the uninstall process of the extension, but it should definitely do some cleanup.

i see no options on the dropdown https://i.imgur.com/tDNbl6F.png I want to just show tweets nothing else to construct a tweetdeck like experience using multiple tabs and no clutter us Vivaldi tiling feature. never mind found it

Currently, the extension is passing CSS variables to the document.body. This is fine but it's a little brittle.

Maybe switch over to creating a selector map in the JavaScript and set styles directly where possible?

It would be nice if the pop up could style itself dependant on the users display settings.

document.body.style.backgroundColor will reveal the users display settings based on that value. Can save this to sync storage and access it within the popup scripts to inject styles into the head 👍

@jh3y

The extension uses MutationObserver which feels a little brittle to hook onto available parts of the UI.

• Is there a better way to do this? Currently, we wait until the DOM is hydrated and then attach attributes so we can get hold of things in the DOM.
• Could we swap out some selectors perhaps?

If you scroll with an adjusted margin, the scrolling will sometimes jump around a bit 😕

This is expected maybe when first adjusting but once scrolled.

@jh3y

I love the extension, great work!

It should also work on mobile.twitter.com. Could you add it to the list of allowed URLs?

• Implement hide DM tray option
• Be wary that this shouldn't use display: none. See here

The containers above the Tweet container creates annoying or bad looking bars. It will look better if you would remove it.