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

Style fixes: Onboarding #11241

Merged
merged 37 commits into from Sep 26, 2019
Merged

Style fixes: Onboarding #11241

merged 37 commits into from Sep 26, 2019

Conversation

negue
Copy link
Member

@negue negue commented Jun 27, 2019

Fixes:

  • Move Justin above the dialog for the last 2 pages
  • fix avatar position
  • item white background + border radius
  • Menu Indicator
  • New Option Style

Extract Parts of Onboarding:

  • Sub-Menu
  • Customize-Option List (Generic for all Items)
  • Body-Settings
  • Skin-Settings
  • Hair-Settings
  • Extras-Settings

Misc.:

  • Moved Tasks to content
    • Replace Strings for Translation

@negue
Copy link
Member Author

negue commented Jun 29, 2019

@SabreCat @Alys could you take a look too?

@Alys
Copy link
Contributor

Alys commented Jun 30, 2019

@negue I can put this on my local install to do some testing. Could you fix the conflict first though?

@negue
Copy link
Member Author

negue commented Jun 30, 2019

As of right now only the menu changed for the body settings (need to replace the submenu on the other pages too), some other styles changed too, but still not finished

As for the merge conflict ... took some time to merge this huge file ^^ one reason more to extract multiple parts of it 😄

@Alys ready for the first look

@Alys
Copy link
Contributor

Alys commented Jul 2, 2019

I'm not super-familiar with the requirements for this but it looks okay.
The top of Justin's head is cropped (or maybe hidden under the header) in some screens like the one below. Not sure if that's intentional or not.

image

@negue
Copy link
Member Author

negue commented Jul 2, 2019

Yeah it shouldn't be under the toolbar, its weird how some browsers work as expected and some not...

What browser / os ?
Was it with client:dev or with the prod-build version ? (there was already an issue I fixed in this PR, which had this kind of dev/prod weirdness)

@Alys
Copy link
Contributor

Alys commented Jul 2, 2019

npm run client:dev
Firefox Quantum 66 on Ubuntu

@paglias
Copy link
Contributor

paglias commented Jul 9, 2019

@negue what's the status on this? also looks like there are some lint issues

@negue negue changed the title [WIP] Style fixes: Onboarding Style fixes: Onboarding Jul 31, 2019
@negue
Copy link
Member Author

negue commented Jul 31, 2019

@Alys I can't reproduce this issue on mac, on any browser, could you check if a simple z-index change would help there?

@Alys
Copy link
Contributor

Alys commented Aug 6, 2019

@negue The head is still being cut off on Firefox on Ubuntu when viewing the delta site. It does look like a z-index problem but I haven't had time to investigate the code to be sure. It's only a display issue though, and it's only for the image. The actual text is still visible and the controls still work.

@paglias
Copy link
Contributor

paglias commented Sep 1, 2019

What's the status on this?

@negue
Copy link
Member Author

negue commented Sep 1, 2019

I'll fix the conflicts

@Tressley anything left todo on this? (In case I missed something)

…i/onboarding

# Conflicts:
#	website/server/models/task.js
@Tressley
Copy link
Collaborator

Tressley commented Sep 9, 2019

@negue -- I'm seeing a bunch of onboarding-related issues on Delta at the moment. Might need to double back to see what is causing the new issues.

@paglias
Copy link
Contributor

paglias commented Sep 9, 2019

Also there are some conflicts due to a different PR that got merged (#11351)

@paglias
Copy link
Contributor

paglias commented Sep 9, 2019

(I'm marking this as needing work for better organization)

@Tressley
Copy link
Collaborator

Tressley commented Sep 16, 2019

@negue -- Can we add a hover state for .option? Something like background-color: rgba(213, 200, 255, .32);? It would look like this:

image

The rest looks good to me!

@negue
Copy link
Member Author

negue commented Sep 16, 2019

Added hover

@Tressley
Copy link
Collaborator

@negue -- Hover doesn't need to be on the item that is selected already.
image

@Tressley
Copy link
Collaborator

This LGTM. 👍

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.

None yet

5 participants