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

fix uib-dropdown according to documentation #418

Merged
merged 1 commit into from
Sep 16, 2019
Merged

fix uib-dropdown according to documentation #418

merged 1 commit into from
Sep 16, 2019

Conversation

bekasovoleg
Copy link

Fix issue #416

@miq-bot
Copy link
Member

miq-bot commented Sep 13, 2019

Checked commit https://github.com/bekasovoleg/ui-components/commit/0bc6130a978b7fc3ee6ae74b5ba8c670f3d6c744 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@bekasovoleg
Copy link
Author

1st dropdown on screenshot has uib-dropdown-menu directive and works properly

https://github.com/ManageIQ/ui-components/blob/master/src/common/components/miqPfSort.html#L7

miq_table

@himdel
Copy link
Contributor

himdel commented Sep 16, 2019

LGTM, thanks! :)


Looks like the original code only worked with angular-ui-bootstrap pre-1.0 (class support was removed in angular-ui/bootstrap#4975).

angular-patternfly dropped 0.14 in patternfly/angular-patternfly@ff80d4a, but that's v4, we're using v3, which still depends on 0.14.

Both ui-classic and ui-service are now using the last angular-ui-bootstrap release - 2.5.6, and have been for about 2 years:

@himdel himdel merged commit 5d4b3ff into ManageIQ:master Sep 16, 2019
@himdel
Copy link
Contributor

himdel commented Sep 16, 2019

We do have css for .uib-dropdown-menu (and .uib-dropdown-menu-right), so looks like the class has to stay too 👍.

Searching for more, 'class='\''[^'\'']*uib' or 'class="[^"]*uib' in ui-service doesn't find anything,
in ui-components, this finds only:

src/toolbar/components/toolbar-menu/toolbar-list.html
2:  <button type="button" uib-dropdown-toggle class="btn uib-dropdown-toggle btn-default"
10:  <ul class="uib-dropdown-menu" uib-dropdown-menu role="menu">

src/site-switcher/site-switcher.html
5:  <div uib-dropdown-menu class="uib-dropdown-menu uib-dropdown-menu-right" aria-labelledby="domain-switcher">

And in ui-classic, \.uib also finds nothing.

So, looks like found the last of these 👍

@himdel himdel self-assigned this Sep 16, 2019
@himdel himdel added this to the Sprint 120 Ending Sep 16, 2019 milestone Sep 16, 2019
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