Skip to content

allow titlefield to be set to false#5496

Open
stuartromanek wants to merge 1 commit into
mainfrom
array-fields-titlefield-false
Open

allow titlefield to be set to false#5496
stuartromanek wants to merge 1 commit into
mainfrom
array-fields-titlefield-false

Conversation

@stuartromanek

Copy link
Copy Markdown
Member

Arrays default back to using title as the default titleField with no way to opt out. My title field was an object and using it as a string was obviously breaking UI

@stuartromanek
stuartromanek requested a review from myovchev June 29, 2026 15:12

@BoDonkey BoDonkey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Never mind - edited because I read the note up top.

@BoDonkey
BoDonkey self-requested a review June 29, 2026 15:28
@BoDonkey

Copy link
Copy Markdown
Contributor

Does the numbered index get automatically added to the UI?

@boutell boutell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd classify it as "minor" but no problem with adding the feature. Needs to make it to documentation as well

? null
: (this.field.titleField || 'title');
const item = this.items.find(item => item._id === id);
return get(item.schemaInput.data, titleField) || `Item ${index + 1}`;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Be sure that get(data, null) doesn't panic.

@myovchev myovchev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A nice tiny feature. I agree it's a minor release (patch is a bug fix usually).

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.

4 participants