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

[Feature] Force favorite item to start new row. #953

Closed
3 of 5 tasks
mk-pmb opened this issue Jun 26, 2022 · 6 comments
Closed
3 of 5 tasks

[Feature] Force favorite item to start new row. #953

mk-pmb opened this issue Jun 26, 2022 · 6 comments
Labels
enhancement New feature or request status: denied This suggestion is denied.

Comments

@mk-pmb
Copy link

mk-pmb commented Jun 26, 2022

What is your feature request?

First up, not a duplicate of categories. While it might be used as a crude stopgap for #388, I'd like to actually use this feature even inside a category.

This feature is a flag that a favorites item might have which forces it to start a new row.

In case explicit visual indication is desired, we could add a thin horizontal line above the icon. I don't need it though.

In order to save on key bindings, I'd suggest we change the "delete existing favorite" key binding to "manage existing favorite", opening a context menu like the "+" menu, where we can view and toggle this flag, can choose to delete, and potentially other actions. (I have a few ideas what other favo actions would be useful.)

[Spoiler / Reminder to myself]
  • renaming without the need for an anvil, because "add favorite" doesn't work in the anvil screen.

What do you think this change is of?

  • Visual
  • Recipe Lookup
  • Cheat Mode
  • Plugin Integration / JEI Plugin Compatibility
  • Others
@mk-pmb
Copy link
Author

mk-pmb commented Jun 26, 2022

In the categories issue, vertical distance was requested ("separated by empty spaces about 2x the item's height."), so maybe we should consider that direction of feature creep here as well. Instead of a flag, it could be an unsigned small integer property. If the property is present, "0" would be the original effect, just force a new line, and any positive number would be how many blank rows to insert.

@shedaniel
Copy link
Owner

I don't think this is a good solution, and I think we should go straight to favorites categories, and not make some middle ground features to be removed when categories finally arrive (Since implementing this also takes time)

@shedaniel shedaniel added the status: denied This suggestion is denied. label Dec 12, 2022
@shedaniel shedaniel closed this as not planned Won't fix, can't repro, duplicate, stale Dec 12, 2022
@mk-pmb
Copy link
Author

mk-pmb commented Dec 12, 2022

@shedaniel When you write,

we should go straight to favorites categories, and not make some middle ground features

I wonder if there was a miscommunication. I don't see this as a middle ground. I knew this could be mistaken, that's why I added

I'd like to actually use this feature even inside a category.

@shedaniel
Copy link
Owner

shedaniel commented Dec 13, 2022

I'd like to actually use this feature even inside a category.

Sorry I don’t actually think it is an user friendly option, yes, power users may love it, but it is so niche. I don’t think you need this level of precise control, this will only confuse users

@mk-pmb
Copy link
Author

mk-pmb commented Dec 13, 2022

Would you be willing to merge and maintain the PR if it hides this feature behind a secret option? E.g. you have to manually edit the JSON to insert the first forced linebreak, but once it's detected on config read, it will be offered.

@shedaniel
Copy link
Owner

It is way too much code and special cases for just that, these code all need maintenance, and I will also need to support that when I actually do favorites groups.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request status: denied This suggestion is denied.
Projects
None yet
Development

No branches or pull requests

2 participants