mattreaganmozilla

@tisumi99 Just wanted to let you know I haven't forgotten about this PR, I'm waiting to get confirmation on the UI changes with our UX team which is something we...

@tisumi99 Still waiting for confirmation from UX team (they like to review UI changes whenever possible before merging). That process can sometimes take a while, but I'll update as soon...

@tisumi99 Apologies for the ongoing delays, I'm still working on getting confirmation from our UX team. PR is still on my radar.

Thanks again @tisumi99 for this PR, merging now.

What I’ve been able to see in testing so far is that we have code which is attempting to fetch the favicon at an incorrect URL (the full website url...

@dataports @OrlaM This is one of those fixes that seems too simple so I'm wondering if I'm missing something. But I tested all of the basic scenarios I could think...

Thanks @ih-codes will take a look.

@ih-codes I can't repro this so far, have tried in a couple different scenarios and device types. Can you please double-check if there's anything missing from the repro steps (or...

> Do you have TabTrayFlagManager.isRefactorEnabled returning true? I only see this behaviour when the tab tray refactor is enabled. Everything works as expected when it's false. @ih-codes I do have...

@ih-codes Ok, so I can repro on your branch, and I can also repro on this branch if I rebase against `main`. It looks like this branch was behind and...