muffon icon indicating copy to clipboard operation
muffon copied to clipboard

Tab shows different content when switching from another tab.

Open dmitry-buzzwoo opened this issue 1 year ago • 19 comments

https://github.com/staniel359/muffon/assets/140138846/7908019a-be44-46ad-b312-56501f2ff6c3

dmitry-buzzwoo avatar Jan 24 '24 04:01 dmitry-buzzwoo

@dmitry-buzzwoo Hello. This is very weird. Added to planned.

staniel359 avatar Jan 24 '24 06:01 staniel359

@staniel359 this is a memory infrastructure bug. I don’t think you’ll be able to solve this. Please asssign me this one.

xyloflake avatar Jan 24 '24 15:01 xyloflake

@dmitry-buzzwoo does this always happen? Are there specific repro steps?

xyloflake avatar Jan 24 '24 15:01 xyloflake

@xyloflake Why is it a memory issue? I assume it's a Vue's cached pages bug.

staniel359 avatar Jan 24 '24 15:01 staniel359

@staniel359 please explain the logic used for keeping tabs separate on muffon in discord DMs.

I suspect this is a memory issue because there are some memory leaks I recently discovered in the code responsible for BrowserView.

xyloflake avatar Jan 24 '24 17:01 xyloflake

@xyloflake I had this problem when I switched from YouTube playlist tab to a video tab. This is the only details I can tell you about.

dmitry-buzzwoo avatar Jan 25 '24 03:01 dmitry-buzzwoo

Also, another unrelated issue is when I use hot key "cmd+r" to update the tab, all tabs are being updated.

dmitry-buzzwoo avatar Jan 25 '24 03:01 dmitry-buzzwoo

@xyloflake I had this problem when I switched from YouTube playlist tab to a video tab. This is the only details I can tell you about.

Thanks. @staniel359 @gamersi can you guys try reproducing the bug?

xyloflake avatar Jan 25 '24 04:01 xyloflake

Also, another unrelated issue is when I use hot key "cmd+r" to update the tab, all tabs are being updated.

That's expected. @staniel359 uses BrowserView to implement tabs. They're not in reality tabs, they're just Out Of Process Iframes. That means all the tabs exist in a single webpage. When you hit Ctrl/Cmd+R, you're triggering the whole webpage to reload, thus reloading all the tabs in existence.

xyloflake avatar Jan 25 '24 04:01 xyloflake

I see, but I suppose in theory it's possible? There is a "refresh" button which reloads just an active tab. I think it's possible to track the currently active tab on "cmd+r" interception and refresh the only active tab like the "refresh" button does?

That's up to you. But I think it will just improve UX.

dmitry-buzzwoo avatar Jan 25 '24 04:01 dmitry-buzzwoo

btw, refreshing the tab that shows an incorrect content (video instead of playlist) doesn't help. To show the correct content, I have to switch to another tab first and then switch back.

dmitry-buzzwoo avatar Jan 25 '24 05:01 dmitry-buzzwoo

@dmitry-buzzwoo Page reloading shouldn't work via shortcuts. Could you send a video of this behavior?

staniel359 avatar Jan 25 '24 07:01 staniel359

https://github.com/staniel359/muffon/assets/140138846/597290f9-8d80-4c3e-949f-4c422aa97198

dmitry-buzzwoo avatar Jan 25 '24 09:01 dmitry-buzzwoo

@staniel359 what's with the horrendous loading speed?

xyloflake avatar Jan 26 '24 08:01 xyloflake

@xyloflake It's basically 6 Electron instances loading at once.

staniel359 avatar Jan 26 '24 09:01 staniel359

@dmitry-buzzwoo may I know your mac specs please?

xyloflake avatar Jan 26 '24 14:01 xyloflake

@xyloflake sure:

Model Name: MacBook Air Model Identifier: Mac14,2 Chip: Apple M2 Total Number of Cores: 8 (4 performance and 4 efficiency) Memory: 16 GB System Firmware Version: 10151.61.4 OS Loader Version: 10151.61.4 System Version: macOS 14.2.1 (23C71) Kernel Version: Darwin 23.2.0 Boot Volume: Macintosh HD Boot Mode: Normal

dmitry-buzzwoo avatar Jan 29 '24 03:01 dmitry-buzzwoo

@staniel359 this is too slow for such a powerful machine.

xyloflake avatar Jan 29 '24 18:01 xyloflake