Skip to content

[hypertable] Updated row height#250

Merged
nathalieArnoux merged 4 commits intomasterfrom
na/dra-3513
Sep 10, 2025
Merged

[hypertable] Updated row height#250
nathalieArnoux merged 4 commits intomasterfrom
na/dra-3513

Conversation

@nathalieArnoux
Copy link
Contributor

What does this PR do?

Related to : #dra-3513

What are the observable changes?

🧑‍💻 Developer Heads Up

⚡ Since we are using Ember Octane now:

  • Feel free to migrate existing components to Glimmer Components.
  • Write new ones exclusively in it.

Useful Resource : Ember Octane vs Classic Cheat Sheet

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated tests
  • Added/updated documentation
  • Migrated touched components to Glimmer Components
  • Properly labeled

Additional Notes

@nathalieArnoux nathalieArnoux self-assigned this Sep 1, 2025
@nathalieArnoux nathalieArnoux requested review from a team and phndiaye as code owners September 1, 2025 15:49
@nathalieArnoux nathalieArnoux requested review from aprentout and olxmpe and removed request for a team September 1, 2025 15:49
@linear
Copy link

linear bot commented Sep 1, 2025

get initials() {
return (
(get(this.args.item, this.args.column.labels[0]) || '')
.match(/\b[a-zA-Z]/g)
Copy link
Contributor

Choose a reason for hiding this comment

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

Couldn't we use nameInitials helper from ember-upf-utils?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was considered, but the helper is not handling special characters correctly and created issues with certain profile names :/

@nathalieArnoux nathalieArnoux merged commit 8a24913 into master Sep 10, 2025
3 checks passed
@nathalieArnoux nathalieArnoux deleted the na/dra-3513 branch September 10, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

Comments