DinoDevs / GladiatusCrazyAddon

This was "A browser addon for the Gladiatus browser game."
https://gladiatus.dinodevs.com
GNU General Public License v3.0
41 stars 30 forks source link

Menu categories - merge all merchants & merge all items into one #430

Closed FrutyX closed 1 year ago

FrutyX commented 1 year ago

just-do-it-shia-la-beouf

GramThanos commented 1 year ago

Just an FYI, modifing the page layout may impact the user experience negatively. Consider the following example:

This is getting worse the slower/older the PC of the user is. For this reason it is suggested not to alter anything on the page that may affect the possitioning os commonly used items souch as menu links. If you still want to do so, the changes should be made by the CSS (or atleast a quick CSS change to set a place holder so that the position do not change after the JavaScript loads, even if the content is not there yet). A similar usecase is been implemented when moving the top event banner, where the CSS class to move the layout is been applied before the page is ready.

FrutyX commented 1 year ago

I completely understand that, and if it's possible, I always use CSS rather than javascript.

However, keep in mind that these features are optional, if the user does not like it, he can always disable it (it's false by default) and all possible changes will be turned off.

GreatApo commented 1 year ago

However, keep in mind that these features are optional, if the user does not like it, he can always disable it (it's false by default) and all possible changes will be turned off.

I completely agree. I would leave it by default off and let people enable it if they want to.

FrutyX commented 1 year ago

I was thinking that maybe I could name those buttons with merchant names (and disable overflow), but I could not find anything in GCA about cached merchant names, they are not used. I only found them in the merchant search, but they are not cached (I think), it seems that their names are loaded during the search request.

I haven't done UI translation caching before, so I am not sure how I should save them...

GreatApo commented 1 year ago

I was thinking that maybe I could name those buttons with merchant names (and disable overflow), but I could not find anything in GCA about cached merchant names, they are not used. I only found them in the merchant search, but they are not cached (I think), it seems that their names are loaded during the search request.

The names are in the menu in every page you visit, you don't really need to cache them

FrutyX commented 1 year ago

When they add the dice event menu category, it will mess up the names order... and I haven't been able to lock on the link... just awesome...

FrutyX commented 1 year ago

Lmao, nevermind, already found a solution. See? Every time I say there is a problem, i will instantly find a solution within the first five minutes, haha.

GreatApo commented 1 year ago

Nice!

It will be more helpful to use icons instead of names in the submenu. image

We may need to create a menu group in the settings at some point.

FrutyX commented 1 year ago

Don't have any icons, unless I would create completely new ones from scratch. I was already searching for some icons in the game UI, but I haven't found anything fitting, and item icons are already used by market and auction menu categories, it would look messy.

For now, this was the best option.

FrutyX commented 1 year ago

Of course I could have used emojis like I did with packages shortcuts, but eh, I don't want to overuse emojis too much, plus figuring out a fitting emoji for smelting or magus etc... meh.

FrutyX commented 1 year ago

We may need to create a menu group in the settings at some point.

Done.

GreatApo commented 1 year ago

Done.

Nice rework Fruty!