patternfly / angular-patternfly

This repo contains instructions and the code for a set of Angular 1 components for the PatternFly project.
http://www.patternfly.org/angular-patternfly/
MIT License
123 stars 90 forks source link

Application Launcher dropdown implementation should remove arrow #741

Open lizsurette opened 6 years ago

lizsurette commented 6 years ago

The new design for the application launcher does not include an arrow in the dropdown. See: http://www.patternfly.org/pattern-library/application-framework/masthead/#design

The application launcher design will be updated to reflect this change and the angular code should be updated as well here: http://www.patternfly.org/angular-patternfly/#/api/patternfly.navigation.component:pfApplicationLauncher

dtaylor113 commented 6 years ago

Hi @lizsurette, are you saying the arrow in the Help dropdown is no longer in the dropdown? Is this a Masthead Issue or an issue for the Application Launcher? -thanks

lizsurette commented 6 years ago

Hey @dtaylor113 there is a bit of history on where this bug was identified over the in react repo here: https://github.com/patternfly/patternfly-react/pull/319

The react team is implementing the launcher and it was identified that the arrow (after the user has clicked on the dropdown there is an arrow pointing back up to the icon they clicked on) should no longer be there so we are just trying to remove the arrow where it's already been implemented.

But essentially, the design of the application launcher has changed with the new masthead design. The application launcher design along with the angular implementation needs to be updated to reflect the new masthead design. @mcarrano is adding an issue on the design side.

lizsurette commented 6 years ago

@dtaylor113 with that being said...I don't think we want that arrow next to the help icon either :) Check this base implementation out of the app launcher: http://www.patternfly.org/pattern-library/application-framework/launcher/#code

dtaylor113 commented 6 years ago

ah, ok, in the App Launcher popup, no arrow pointing back up to the App Launcher icon in the masthead. -thanks