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

Tour Fix #616

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Tour Fix #616

wants to merge 1 commit into from

Conversation

jf-kelly
Copy link
Contributor

Description

The tour now shows both the keyboard shortcut button and the font size button. Previously, it only showed the font size button.

Preview

2022-07-28

Related Issue

#598

@jf-kelly jf-kelly requested review from a team, rjmara, banthedev and nkisari and removed request for a team July 28, 2022 20:24
@jf-kelly jf-kelly added Frontend Issues being caused via the frontend UI/UX labels Aug 2, 2022
@rjmara
Copy link
Contributor

rjmara commented Aug 9, 2022

@jf-kelly This change looks good.

I know this isn't part of this PR but I thought it would be helpful to add it here. When I take the tour on dev, the 5th item, the explanation of the save button, highlights only half of the button. I'm on a windows laptop using chrome so I'm not sure if it's a problem for everybody but It's something worth checking to see if others have the same problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Issues being caused via the frontend UI/UX
Projects
Status: Needs Review
Development

Successfully merging this pull request may close these issues.

2 participants