Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
FrutyX opened this issue Oct 15, 2022 · 12 comments
Closed

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

FrutyX opened this issue Oct 15, 2022 · 12 comments

Comments

@FrutyX
Copy link
Collaborator

FrutyX commented Oct 15, 2022

just-do-it-shia-la-beouf

@GramThanos
Copy link
Member

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

  • The page loads and the user instantly scrolls down to click the auction button.
  • While his cursor is above the auction button, the addon loads and changes the layout
  • The user clicked the right mouse button thinking he will end up at the auction
  • Due to the layout change the auction button is now a few pixels up and the user ends up clicking the market button.

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
Copy link
Collaborator Author

FrutyX commented Oct 15, 2022

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
Copy link
Member

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 added a commit that referenced this issue Oct 15, 2022
@FrutyX
Copy link
Collaborator Author

FrutyX commented Oct 15, 2022

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
Copy link
Member

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 added a commit that referenced this issue Oct 19, 2022
FrutyX added a commit that referenced this issue Oct 19, 2022
FrutyX added a commit that referenced this issue Oct 20, 2022
@FrutyX FrutyX changed the title Menu categories - merge all merchants into one & merge all items into one Menu categories - merge all merchants & merge all items into one Oct 21, 2022
@FrutyX
Copy link
Collaborator Author

FrutyX commented Oct 21, 2022

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
Copy link
Collaborator Author

FrutyX commented Oct 21, 2022

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.

FrutyX added a commit that referenced this issue Oct 21, 2022
@GreatApo
Copy link
Member

GreatApo commented Oct 23, 2022

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
Copy link
Collaborator Author

FrutyX commented Oct 23, 2022

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
Copy link
Collaborator Author

FrutyX commented Oct 23, 2022

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
Copy link
Collaborator Author

FrutyX commented Oct 29, 2022

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

Done.

@GreatApo
Copy link
Member

Done.

Nice rework Fruty!

@FrutyX FrutyX removed the ⌛ testing label Nov 9, 2022
@FrutyX FrutyX closed this as completed Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants