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

Minor UI issue with Idea Hub icon on 'View only' menu #5446

Closed
eclarke1 opened this issue Jun 27, 2022 · 4 comments
Closed

Minor UI issue with Idea Hub icon on 'View only' menu #5446

eclarke1 opened this issue Jun 27, 2022 · 4 comments
Labels
Good First Issue Good first issue for new engineers P1 Medium priority Type: Bug Something isn't working

Comments

@eclarke1
Copy link
Collaborator

eclarke1 commented Jun 27, 2022

Bug Description

Bug bash issue: https://app.asana.com/0/1202258919887896/1202419867342278 please see Asana for background

The idea hub icon is slightly larger and unaligned with the text. Only minor.
image.png


Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

  • The module icon, name, and "shared by" info should all be vertically aligned/centered in the "view only" Dashboard Sharing menu.

Implementation Brief

Test Coverage

  • Update VRT screenshots if required.

QA Brief

  • Ensure that the dashboardSharing feature flag is enabled in the tester plugin.
  • Go to the Site Kit Dashboard.
  • Using the Dashboard Sharing settings modal, grant access to some/all modules to one or more roles, which should include the Idea Hub module.
  • Log into the site using a different user having one of the granted roles.
  • Open the Site Kit Dashboard as a View Only user.
  • Open the View Only menu.
  • Ensure that the Idea Hub icon and all the other icons are correctly placed.

Changelog entry

  • Improve "View only" menu icon alignment when viewing shared dashboard.
@eclarke1 eclarke1 added P1 Medium priority Type: Bug Something isn't working labels Jun 27, 2022
@tofumatt tofumatt assigned tofumatt and unassigned tofumatt Jun 28, 2022
@tofumatt tofumatt added the Good First Issue Good first issue for new engineers label Jun 28, 2022
@nfmohit nfmohit assigned nfmohit and unassigned nfmohit Jun 29, 2022
@aaemnnosttv aaemnnosttv self-assigned this Jun 29, 2022
@aaemnnosttv
Copy link
Collaborator

  • Update the <Icon /> component here to have a height prop with the value of 26 only if it is the idea-hub module:

@nfmohit why not set the same height consistently for all module icons? It seems odd that Idea Hub would be a special case here.

@aaemnnosttv aaemnnosttv assigned nfmohit and unassigned aaemnnosttv Jun 29, 2022
@nfmohit
Copy link
Collaborator

nfmohit commented Jun 30, 2022

@aaemnnosttv You're right! I just double-checked and found out that applying the height for all module icons doesn't make a difference to their current dimensions (except Idea Hub, of course, 😅). My bad 🙏. I've updated the IB to apply it for all module icons.

Thank you so much!

@nfmohit nfmohit assigned aaemnnosttv and unassigned nfmohit Jun 30, 2022
@aaemnnosttv
Copy link
Collaborator

IB ✅

@aaemnnosttv aaemnnosttv removed their assignment Jul 1, 2022
@nfmohit nfmohit self-assigned this Jul 5, 2022
@nfmohit nfmohit removed their assignment Jul 6, 2022
@tofumatt tofumatt assigned tofumatt and unassigned tofumatt Jul 6, 2022
@wpdarren wpdarren self-assigned this Jul 11, 2022
@wpdarren
Copy link
Collaborator

QA Update: ✅

Verified:

  • the Idea Hub icon and all the other icons are correctly placed.

image

Note: I am struggling to see the idea hub icon because of the grey background on the view only menu. I have raised this as a GM2+ issue as it's out of scope for this ticket.

@wpdarren wpdarren removed their assignment Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Good first issue for new engineers P1 Medium priority Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants