[Feature] Force favorite item to start new row.
mk-pmb opened this issue · 6 comments
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
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.
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 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.
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
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.