Skip to content

Display first field instead of note ID#148

Merged
tatsumoto-ren merged 3 commits into
Ajatt-Tools:masterfrom
emiham:card-info
Dec 2, 2025
Merged

Display first field instead of note ID#148
tatsumoto-ren merged 3 commits into
Ajatt-Tools:masterfrom
emiham:card-info

Conversation

@emiham

@emiham emiham commented Sep 11, 2025

Copy link
Copy Markdown
Contributor

I don't think showing the note ID is very useful. When updating cards it's a lot more useful to get a readable representation of the card, so you know that you actually updated the right card.

Right now it just uses the first field, but I could also see this as a config option. In theory there could be a toggle for if you want to show field contents or the note ID, but I don't think the note ID is useful enough for that.

@tatsumoto-ren What do you think?

@tatsumoto-ren

Copy link
Copy Markdown
Member

For me it's fine either way. When I update a note, the Browser window shows up. I usually edit the note right after.

@emiham

emiham commented Sep 14, 2025

Copy link
Copy Markdown
Contributor Author

I'll move ahead with this then.

Do you think it should be a config option or is the first field fine? First field requires one less option which I think is nice since there are quite a few already, but there's no guarantee you have something useful in the first field. On the other hand, I still think it's strictly better than showing the ID, so it could be changed to a config option later if needed.

@tatsumoto-ren

Copy link
Copy Markdown
Member

Do you think it should be a config option or is the first field fine?

I think it's unnecessary to create a config option for something as simple as this.

but there's no guarantee you have something useful in the first field

You can check if it's empty and display "empty note" or "note id xxx" in such cases.

@tatsumoto-ren

Copy link
Copy Markdown
Member

Is this ready for review yet?

@emiham
emiham marked this pull request as ready for review December 2, 2025 14:15
@emiham

emiham commented Dec 2, 2025

Copy link
Copy Markdown
Contributor Author

Should be good now, sorry for taking so long.

@tatsumoto-ren
tatsumoto-ren merged commit 5fb048c into Ajatt-Tools:master Dec 2, 2025
@tatsumoto-ren

Copy link
Copy Markdown
Member

Thank you!

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