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

Angular: Adds back two angular directives that are still used by remaining angular bits and plugins #50380

Merged
merged 1 commit into from
Jun 8, 2022

Conversation

torkelo
Copy link
Member

@torkelo torkelo commented Jun 8, 2022

Adds back bs-tooltip and bs-typeahead that where removed when I removed the angular-strap file

@torkelo torkelo requested review from a team, zoltanbedi and mdvictor and removed request for a team June 8, 2022 08:05
@grafanabot grafanabot added datasource/MSSQL Microsoft SQL Server Data Source area/frontend labels Jun 8, 2022
@torkelo torkelo requested review from jackw and removed request for zoltanbedi and mdvictor June 8, 2022 08:06
@torkelo torkelo added this to the 9.0.0 milestone Jun 8, 2022
Copy link
Contributor

@alexanderzobnin alexanderzobnin left a comment

Choose a reason for hiding this comment

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

LGTM!

<h3 class="page-heading">MS SQL connection</h3>

<div class="gf-form-group">
<div class="gf-form max-width-30">
<span class="gf-form-label width-7">Host</span>
<input type="text" class="gf-form-input" style="width: 352px" ng-model='ctrl.current.url' placeholder="localhost" bs-typeahead="{{['localhost', 'localhost" required></input>
Copy link
Member Author

Choose a reason for hiding this comment

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

this line is the only one that changed, the bs-typeahead line was an incomplete expression. Then prettier formatted the whole document :)

Copy link
Contributor

@jackw jackw left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@torkelo torkelo merged commit 288eede into main Jun 8, 2022
@torkelo torkelo deleted the add-back-angular-directives branch June 8, 2022 08:34
grafanabot pushed a commit that referenced this pull request Jun 8, 2022
…ining angular bits and plugins (#50380)

(cherry picked from commit 288eede)
return value;
},
});
var typeahead = element.dat('typeahead');
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be

var tooltip = element.data('tooltip');

torkelo added a commit that referenced this pull request Jun 8, 2022
…ining angular bits and plugins (#50380) (#50385)

(cherry picked from commit 288eede)

Co-authored-by: Torkel Ödegaard <torkel@grafana.com>
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.

4 participants