Skip to content

Conversation

@mlell
Copy link
Contributor

@mlell mlell commented Dec 17, 2025

It might be just my taste, but for my uses the new entry UI has some papercuts:

  • Payee field is large and narration field is small, despite my payees are seldom more than 3 words and the narration is long
  • Metadata value fields are very short. I keep things there like the original narration (like it is on my bank statement => long) or a merged string of all SEPA identifiers
  • Buttons disappear on mouse hover, which makes it unusable for touch screen users.

This PR fixes that, I have added before/after screencasts below:

Before:

Bildschirmaufzeichnung.vom.2025-12-17.16-21-20.mp4

After:

Bildschirmaufzeichnung.vom.2025-12-17.16-22-18.mp4

Copy link
Member

@yagebu yagebu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

}

div :global(.narration) {
flex-grow: 5;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This tilts the balance too far in the other direction IMHO. This now leads to most of my payees being cut off

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed to flex-grow: 2,

Before:
grafik

After:

grafik

@mlell mlell force-pushed the dev_entry_ui_access branch from 9e69e32 to 47b3d88 Compare December 18, 2025 10:31
@yagebu
Copy link
Member

yagebu commented Dec 30, 2025

@mlell I've made some more wide-ranging refactors to these components and incorporated your improvements as well in #2169, can you take a look at that one?

@yagebu
Copy link
Member

yagebu commented Jan 2, 2026

Thanks, these changes have been applied with #2169

@yagebu yagebu closed this Jan 2, 2026
@mlell mlell deleted the dev_entry_ui_access branch January 4, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants