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

(PXP-7097) Add optional alternate TopBar profile menu #759

Merged
merged 4 commits into from
Nov 16, 2020

Conversation

olivarity
Copy link
Contributor

Jira Ticket: PXP-7097

New Features

  • Added new dropdown menu to TopBar component for handling user profile and logout options, opt-in via config components.topBar.useProfileDropdown: true

Deployment changes

  • At time of writing, qa-midrc is only env using the new feature flag. These changes should be deployed alongside each other to production, but nothing will break if done separately.

@mfshao
Copy link
Collaborator

mfshao commented Nov 13, 2020

is this thing (and the accompanying CSS changes) deployed anywhere so we can take a look?

@olivarity
Copy link
Contributor Author

is this thing (and the accompanying CSS changes) deployed anywhere so we can take a look?

@mfshao waiting on uc-cdis/gitops-qa#1051 checks to finish, and then we'll deploy this version there :)

@mfshao
Copy link
Collaborator

mfshao commented Nov 13, 2020

is this thing (and the accompanying CSS changes) deployed anywhere so we can take a look?

@mfshao waiting on uc-cdis/gitops-qa#1051 checks to finish, and then we'll deploy this version there :)

great! thanks

it would be easier if you put all these changes locally into your dev env so we can see it without waiting for jenkins

Copy link
Collaborator

@mfshao mfshao left a comment

Choose a reason for hiding this comment

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

tested locally and lgtm in general with some minor comments

src/Layout/reduxer.js Show resolved Hide resolved
isActive={this.isActive('/identity')}
onActiveTab={() => this.props.onActiveTab('/identity')}
/>
</Link>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I thought for this part there were some inconsistence in the design. Have we made a dicision that whether we display only a single icon or icon + username after user has logged in

If we decide to keep only a single icon, can you adjust the placement of the popover so the arrow gets pointed to the icon?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Either option is feasible, we'll have to put that one up to the team. Pushing a commit with correct placement now, that was just a temporary change that slipped through during experimentation.

Copy link
Collaborator

@mfshao mfshao left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@olivarity olivarity merged commit 3fc54c9 into master Nov 16, 2020
@olivarity olivarity deleted the feat/midrc_profile branch November 16, 2020 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants