angular-patternfly icon indicating copy to clipboard operation
angular-patternfly copied to clipboard

Application Launcher dropdown implementation should remove arrow

Open lizsurette opened this issue 7 years ago • 4 comments

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

lizsurette avatar May 24 '18 18:05 lizsurette

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

dtaylor113 avatar May 24 '18 18:05 dtaylor113

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 avatar May 24 '18 19:05 lizsurette

@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

lizsurette avatar May 24 '18 19:05 lizsurette

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

dtaylor113 avatar May 24 '18 19:05 dtaylor113