Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CollapsingToolbarLayout] Allow application to Set ellipsis to Title in CollapsingToolBar #2636

Closed

Conversation

venugopalraog
Copy link
Contributor

@venugopalraog venugopalraog commented Apr 13, 2022

Thanks for starting a pull request on Material Components!

Don't forget:

  • Identify the component the PR relates to in brackets in the title.
    [Buttons] Updated documentation
  • Link to GitHub issues it solves. closes # 2631
  • Sign the CLA bot. You can do this once the pull request is opened.

Contributing
has more information and tips for a great pull request.

Screenshots:
screenshot_middle_ellipsize_collapsed
screenshot_middle_ellipsize_expanded

Copy link

@chaituapps2408 chaituapps2408 left a comment

Choose a reason for hiding this comment

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

Looks good

@venugopalraog venugopalraog changed the title closes #2631 [CollapsingToolbarLayout] Allow application to Set ellipsis to Title in CollapsingToolBar [CollapsingToolbarLayout] Allow application to Set ellipsis to Title in CollapsingToolBar Apr 18, 2022
Copy link
Contributor

@drchen drchen 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 working on this. Will send the change for our internal review. : )

@drchen drchen self-assigned this Apr 19, 2022
@venugopalraog
Copy link
Contributor Author

Thanks for working on this. Will send the change for our internal review. : )

@drchen Thank you for update. Any update from internal review..

@drchen
Copy link
Contributor

drchen commented Apr 27, 2022

Hey, sorry for the delay. I need to add internal tests and doc updates. Hopefully will get this through this week or next week.

@dsn5ft dsn5ft closed this in 7b043c3 Apr 28, 2022
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.

None yet

3 participants