search

lindylearn / unclutter

A modern reader mode and article library for your browser.
https://unclutter.lindylearn.io
GNU Affero General Public License v3.0
1.25k stars 54 forks source link

Original style broken after disabling Unclutter #25

Open phgn0 opened 2 years ago

phgn0 commented 2 years ago

After disabling Unclutter via the extension icon, the original site often looks broken. (not all style is removed properly)

Multiple users have reported finding this annoying, so we should address this.

phgn0 commented 2 years ago

A fix for this should release early next week.

phgn0 commented 2 years ago

This is now done and should release tomorrow!

phgn0 commented 1 year ago

@bdmbdsm brought this up again on Discord: some complex page layouts have issues after disabling Unclutter on a page. Let's look at the reader mode style undo again.

bdmbdsm commented 1 year ago

@phgn0 Here's the page. Perhaps nobody is going to use unclutter on Notion, because it's readable enough, but it shows that complex pages such Notion are not clearly reset.

Were you able to reproduce the broken undo?

phgn0 commented 1 year ago

Yeah Notion is a special case, they use a lot of JavaScript that listens to DOM events and reverts changes Unclutter does. See https://github.com/lindylearn/unclutter/issues/662

But the undo function should work better regardless. I'll try to look into this!