Skip to content

Conversation

InnaAtanasova
Copy link
Contributor

Description

  • modifier classes for text truncation
  • popover header on scroll when user name is no longer visible
  • updated tests
  • examples update
    IMPORTANT: brought some CSS from fund-styles as we can't release fund-styles not and can't use the RC version

Screenshots

User name is visible (no heading)
Screenshot 2025-10-06 at 2 58 20 PM

User name is not visible (heading)
Screenshot 2025-10-06 at 2 58 10 PM

@InnaAtanasova InnaAtanasova requested a review from a team October 6, 2025 19:54
@InnaAtanasova InnaAtanasova self-assigned this Oct 6, 2025
Copy link

netlify bot commented Oct 6, 2025

Deploy Preview for fundamental-ngx ready!

Name Link
🔨 Latest commit 51be302
🔍 Latest deploy log https://app.netlify.com/projects/fundamental-ngx/deploys/68e548dceaec63000862174c
😎 Deploy Preview https://deploy-preview-13518--fundamental-ngx.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@InnaAtanasova InnaAtanasova force-pushed the feat/user-menu-latest-changes branch 2 times, most recently from b275a94 to 6b3ec52 Compare October 7, 2025 16:30
@InnaAtanasova InnaAtanasova force-pushed the feat/user-menu-latest-changes branch from 6b3ec52 to 51be302 Compare October 7, 2025 17:07
@Input()
@HostBinding('class.fd-list__title--wrap')
wrap = false;
wrap = input(false, { transform: booleanAttribute });
Copy link
Member

Choose a reason for hiding this comment

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

So I know you're just updating this to an input signal. And I know wrap defaults to false and doesn't apply a class. But the default behavior is to wrap. Changing the value of this input doesn't have any actual affect on the text as far as I can tell. Should we deprecate this input?

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.

3 participants