Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

[tooltip] Add text wrapping in a tooltip #142

Closed
pkozlowski-opensource opened this issue Feb 15, 2013 · 3 comments
Closed

[tooltip] Add text wrapping in a tooltip #142

pkozlowski-opensource opened this issue Feb 15, 2013 · 3 comments
Assignees

Comments

@pkozlowski-opensource
Copy link
Member

Currently text can't be wrapped in a tooltip. The original bootstrap directive allows HTML tags in the tooltips text but for us to support this we would have to either:

The more I think about it the more I'm getting warmed up to the idea of using ngSanitize...
@angular-ui/bootstrap thoughts?

@joshdmiller
Copy link
Contributor

I agree, @pkozlowski-opensource. Though I really don't like having to require ngSanitize, I like the alternative a lot less.

angular-sanitize.js is only 3.8K. For reference, angular.js is around 78K and ui/bootstrap is currently around 20K, so that's only an increase of about 4%.

Unless there are objections (or a better idea), I'll submit a PR this weekend.

@pkozlowski-opensource
Copy link
Member Author

@joshdmiller yes, as much as I hate adding new dependencies I'm pretty much resigned to the idea of including dependency on ngSanitize :-( Unless someone can see another magical solution... Bumped into this once again today while working on the typeahead.

PR would be awesome.

@pkozlowski-opensource
Copy link
Member Author

Duplicate of #246

codedogfish pushed a commit to codedogfish/angular-ui-bootstrap that referenced this issue Sep 15, 2015
Show placeholder when model is empty string
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants