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

Allow edit postings notes #229

Open
advy99 opened this issue Jan 11, 2025 · 5 comments · May be fixed by #230
Open

Allow edit postings notes #229

advy99 opened this issue Jan 11, 2025 · 5 comments · May be fixed by #230
Labels
enhancement New feature or request

Comments

@advy99
Copy link
Contributor

advy99 commented Jan 11, 2025

Since #223 the notes on transactions are stored and not deleted when editing a transaction, but it would be nice if we can add or edit a note on a posting from the app.

Currently I have implemented a simple way to edit notes, but before opening a PR I wanted to know the developer opinion. I've added a button on each PostingRow to show a Dialog with the note. This allows to create/edit/delete a note in a posting.

I am no graphic designer, but if you think this will be a nice feature I can made a PR.

@advy99 advy99 added the enhancement New feature or request label Jan 11, 2025
@chvp
Copy link
Owner

chvp commented Jan 11, 2025

I'm not opposed to adding the functionality of editing comments, but I'm not a big fan of the UX as implemented in that screenshot. I personally feel that the row for postings is already constrained horizontally in size, so adding another button to that will only make that problem worse. I'm also just not a big fan of hiding this behind a dialog.

If/when this is added, each posting should probably be multiple rows. This would also give more space for things like balance assertions, posting status, costs, ...

@alerque
Copy link

alerque commented Jan 11, 2025

Yes, the horizontal size is already an issue even before the extra button. I agree the UI needs a rework—that might even be appreciated without posting notes enabled.

@advy99
Copy link
Contributor Author

advy99 commented Jan 12, 2025

Yeah, I also thought that is constrained horizontally. What do you think about a design that when you select a posting, additional rows associated to that posting appears (for balance assertion, notes, etc). As you say, that way the horizontal space would not be a problem, and also in the settings we can add a check for something like "Simple/Complex UI", just in case somebody just want the basic posting view and does not want/use all of that information

@alerque
Copy link

alerque commented Jan 12, 2025

That was along the lines of what I had in my head as the best idea I could think of. I didn't love it so I didn't bother suggesting it, but also I couldn't think of anything better.

The other idea I was toying with was having just the account on one row and everything else related to the posting in a second row.

@chvp
Copy link
Owner

chvp commented Jan 13, 2025

Something like that sounds good to me. I'm not a UI expert either, I usually just experiment a bit with different layouts and see what I like the best.

@advy99 advy99 linked a pull request Jan 15, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants