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

Top Bar Alignment and Color Issues #49154

Closed
noahtallen opened this issue Jan 21, 2021 · 11 comments
Closed

Top Bar Alignment and Color Issues #49154

noahtallen opened this issue Jan 21, 2021 · 11 comments
Assignees
Labels
[Feature] Calypso & wp-admin Navigation All navigation in Calypso and wp-admin, and the unified transitions between the two. [Pri] Low Address when resources are available. [Type] Bug When a feature is broken and / or not performing as intended

Comments

@noahtallen
Copy link
Contributor

Steps to reproduce the behavior

There are some CSS alignment differences when viewing the topbar in Calypso vs on the front-end of a site (all logged-in views). Some are very significant, and some are pretty slight. I would say the mobile front-end view has the most jarring difference, and the desktop differences are more slight.

The difference in color is very obvious though.

  1. In calypso, the shade of gray for the top-bar background color is darker than the shade of gray on a site frontend.
  2. On front end views, on desktop viewports, there is significantly more left margin on the "reader" button than there is in the Calypso view.
  3. On front-end views, on mobile viewports, there is significantly more left margin beside the "my sites" button than there is in the calypso view.
  4. On front-end views, on mobile viewports, there is more margin between all buttons than there is in calypso.
  5. On front-end views, on desktop viewports, there is more right margin on the profile icon than there is in calypso.
  6. On front-end views, there is no "write" button. This is probably deliberate, unlike the other problems, but it is still a difference.

Desktop Calypso:

Screen Shot 2021-01-21 at 3 01 23 PM

Desktop Front-End:

Screen Shot 2021-01-21 at 3 01 28 PM

Mobile Calypso:

Screen Shot 2021-01-21 at 3 01 52 PM

Mobile Front-End:

Screen Shot 2021-01-21 at 3 01 58 PM

Browser / OS version

All

Is this specific to the applied theme? Which one?

No

Does this happen on simple or atomic sites or both?

All sites

Is there any console output or error text?

Level of impact (Does it block purchases? Does it affect more than just one site?)

All user logged-in views

Screenshot / Video: If applicable, add screenshots to help explain your problem.

See above

@noahtallen noahtallen added the [Type] Bug When a feature is broken and / or not performing as intended label Jan 21, 2021
@obenland obenland added [Feature] Calypso & wp-admin Navigation All navigation in Calypso and wp-admin, and the unified transitions between the two. Nav-Unification [Beta] labels Jan 21, 2021
@getdave getdave added the [Pri] Low Address when resources are available. label Jan 25, 2021
@obenland obenland added this to the Nav Unification [Beta] milestone Mar 10, 2021
@sixhours sixhours self-assigned this Apr 6, 2021
@sixhours
Copy link
Contributor

sixhours commented Apr 6, 2021

It looks like the front-end spacing weirdness issue on desktop is specific to P2; my normal simple sites and atomic sites are fine:

Screen Shot 2021-04-06 at 11 38 27 AM

Screen Shot 2021-04-06 at 11 37 45 AM

On P2, there's an additional invisible menu item in that space, #wp-admin-bar-menu-toggle, that does not exist on normal sites. I'm not sure where it's coming from or what it does (I would have thought it was a mobile menu toggle, but it's invisible on mobile screens, too). In P2's CSS there's a reference to it that seems to suggest even P2 folks don't know what it is:

        // Prevent this invisible toggle from interfering with our
	// mobile menu toggle button when hovering near the edge of the window.
	// (What is this even?)
	#wp-admin-bar-menu-toggle {
		pointer-events: none;
	}

Confirmed this is happening on both internal P2 as well as non-internal P2:
Screen Shot 2021-04-06 at 12 07 40 PM

@sixhours
Copy link
Contributor

sixhours commented Apr 6, 2021

Aha, it's the core menu toggle for /wp-admin:
Screen Shot 2021-04-06 at 11 57 33 AM

I'm not sure why it's showing on P2 sites' front-end but it should be safe to display: none it at the bare minimum.

@sixhours
Copy link
Contributor

sixhours commented Apr 6, 2021

In calypso, the shade of gray for the top-bar background color is darker than the shade of gray on a site frontend.

I think this is the expected behavior. In Calypso, the top bar color will change depending on your color scheme, and those won't be reflected on the front end. We could update the default color scheme top bar color to match the front end for closer visual parity, but I don't think we should set the expectation that the front end will match Calypso.

@sixhours
Copy link
Contributor

sixhours commented Apr 6, 2021

Related issue: #47386

@sixhours
Copy link
Contributor

sixhours commented Apr 6, 2021

Started a PR for the extra left-hand padding in D59796-code

@noahtallen
Copy link
Contributor Author

I don't think we should set the expectation that the front end will match Calypso

I think my personal expectation would be for that bar to match my color scheme (especially if wp-admin/calypso are being kept in sync now) because it's part of the logged-in management UI

Thank you for looking into this! Glad it's not as wide-spread as it seemed at first.

@sixhours
Copy link
Contributor

sixhours commented Apr 7, 2021

I think my personal expectation would be for that bar to match my color scheme (especially if wp-admin/calypso are being kept in sync now) because it's part of the logged-in management UI

@noahtallen That's fair! I've opened a new issue in #51757 since I can see it going both ways and it's not technically a bug.

@sixhours
Copy link
Contributor

sixhours commented Apr 7, 2021

Just breaking this out into checkboxes so I can keep track of what's shipped:

  • In calypso, the shade of gray for the top-bar background color is darker than the shade of gray on a site frontend. (TBD in Top bar: Should front-end match Calypso color scheme? #51757)
  • On front end views, on desktop viewports, there is significantly more left margin on the "reader" button than there is in the Calypso view. (P2 only, reported in Automattic/p2#2443)
  • On front-end views, on mobile viewports, there is significantly more left margin beside the "my sites" button than there is in the calypso view - D59796-code
  • On front-end views, on mobile viewports, there is more margin between all buttons than there is in calypso (Addressed in Top bar: Spacing tweaks to match front end #51720)
  • On front-end views, on desktop viewports, there is more right margin on the profile icon than there is in calypso. (Addressed in Top bar: Spacing tweaks to match front end #51720)
  • On front-end views, there is no "write" button. This is probably deliberate, unlike the other problems, but it is still a difference. (Deliberately hidden on P2 only, asked about it here: Top Bar Alignment and Color Issues #49154 (comment))

@sixhours
Copy link
Contributor

sixhours commented Apr 7, 2021

The absence of a Write button on the front end is also specific to the new P2 (not older versions); normal site:

Screen Shot 2021-04-07 at 2 38 47 PM

P2:

Screen Shot 2021-04-07 at 2 40 11 PM

This was introduced by @mirka about a year ago in Automattic/p2#115, maybe they know why we're hiding it for P2?

@mirka
Copy link
Member

mirka commented Apr 7, 2021

This was introduced by @mirka about a year ago in Automattic/p2#115, maybe they know why we're hiding it for P2?

Right, this was a deliberate design choice to encourage use of the front-end editor. And also to not confuse those who are unfamiliar to WP with two different editors.

@kwight
Copy link
Contributor

kwight commented Apr 14, 2021

All of the user-facing issues have been fixed – there's one remaining P2-specific issue being tracked and handled by that team. Closing.

@kwight kwight closed this as completed Apr 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Calypso & wp-admin Navigation All navigation in Calypso and wp-admin, and the unified transitions between the two. [Pri] Low Address when resources are available. [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Development

No branches or pull requests

6 participants