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

merge distinct items in list =enhancement #26

Open
andr222 opened this issue Nov 28, 2022 · 4 comments
Open

merge distinct items in list =enhancement #26

andr222 opened this issue Nov 28, 2022 · 4 comments
Labels

Comments

@andr222
Copy link

andr222 commented Nov 28, 2022

Current version 6.1.0
This is a common function on other note apps with lists.
It is usually implemented with simple backspace/delete.

It can be worked around, but it would be very convenient, and should be easy to implement.

@andr222 andr222 changed the title merge distinct items in list merge distinct items in list =enhancement Nov 28, 2022
@federicoiosue
Copy link
Owner

Hi there,

as you stated yourself the same goal is achievable by backspacing text: that's a great example of K.I.S.S. approach where no specific function is needed because the most general one just works.
Not having nested lists or complex structures I'd keep things as they are now in relation to this request.

@andr222
Copy link
Author

andr222 commented Feb 13, 2023

In order to merge 2 adjacent elements in a list, one must

  1. desactivate the list,
  2. remove the return between the 2 items to be merged,
  3. re-activate the list.

Being able to simply backspace on the beginning of the second of the items to be merged, while keeping the list activated, would be a plus.
(Like some other note apps.)

I suppose the current functioning could be considered a feature, since it isn't possible to accidently merge 2 items, but for me that isn't a problem.

@federicoiosue
Copy link
Owner

Got it, actually it used to work exactly like that time ago, maybe something changed and this behavior has not been verified again.

@federicoiosue federicoiosue reopened this Feb 17, 2023
@federicoiosue federicoiosue transferred this issue from federicoiosue/Omni-Notes Feb 17, 2023
@federicoiosue
Copy link
Owner

@andr222 I've moved it here because this is the checklist library used to get that feature into Omni Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants