Show full server info for frequent and recent locations

[hankolsen]

Fixes #364

Allow Button component to use whitespace-normal class so the text can wrap if needed.
Update MostUsedLocationButtons and RecentLocationButtons to not truncate text and use whitespace-normal.
Update MostUsedLocationButtons and RecentLocationButtons to center the trash icon vertically.

[ruihildt]

I was also thinking of removing from the names wg-socks5 so for example changing al-tia-wg-socks5-001 to al-tia-001. Since it doesn't give actual information, and the context makes it pretty clear that this is proxy server we're dealing with.

Also relaized, we're not being consistent across the different places:

But maybe that's for another PR?

[hankolsen]

Updated the Location component:

[ruihildt]

found an unwanted side-effect in terms of buttons, and a missing update in terms of naming:

[hankolsen]

1. I don't see that line break. Weird. (For me it has the whitespace-nowrap class
   <button class="btn whitespace-nowrap bg-blue text-white btn-small">Select location</button>)
2. Would you like wg- to be removed here? (I've only removed mentions of wg-socks5-so far)

[ruihildt]

• I don't see that line break. Weird.

It's weird, I can reproduce it only half reliably.

• Would you like wg- to be removed here? (I've only removed mentions of wg-socks5-so far)

Yes, so it's the same information everywhere.

[hankolsen]

It's weird, I can reproduce it only half reliably.

When you can reproduce, could you check to see if the button has the class whitespace-nowrap or not?

[ruihildt]

When you can reproduce, could you check to see if the button has the class whitespace-nowrap or not?
It does.

[hankolsen]

Looks like the class definition is missing in your version...
Restart the dev server?

[ruihildt]

Looks like the class definition is missing in your version... Restart the dev server?
Restarting doesn't help in those cases, I wonder if this is another case of npm vs pnpm.

Here's a binary I created, could you try it on your end and check if you also see it? (needs renaming from .txt to .xpi)

mullvad-browser-extension-0.9.11.txt

[hankolsen]

Found the issue with the missing class:
https://unocss.dev/guide/extracting#safelist

[ruihildt]

Nice, we're nearly there.^^"

In getRandomSessionProxy.ts (line 42) and useSocksProxy.ts (line 230), there are still leftover of the old hostname renaming, which can go entirely thanks to your change at the source in the flatProxiesList.

[hankolsen]

Nice, we're nearly there.^^"

In getRandomSessionProxy.ts (line 42) and useSocksProxy.ts (line 230), there are still leftover of the old hostname renaming, which can go entirely thanks to your change at the source in the flatProxiesList.

CTRL + F failed me…

[hankolsen]

Fixed!

[ruihildt]

Whoop whoop.