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

improve(home): interactive scheme #520

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

unordinarity
Copy link
Member

Changelog

replaced static scheme in home page with interactive one

@netlify
Copy link

netlify bot commented Aug 15, 2022

Deploy Preview for pr-sliced ready!

Name Link
🔨 Latest commit 453e064
🔍 Latest deploy log https://app.netlify.com/sites/pr-sliced/deploys/62fa3e59ede5fd0008366b18
😎 Deploy Preview https://deploy-preview-520--pr-sliced.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@illright
Copy link
Member

image

Something doesn't seem right here

@unordinarity
Copy link
Member Author

unordinarity commented Aug 30, 2022

@illright

Something doesn't seem right here

What a pain, I forgot we have two color themes

@azinit
Copy link
Member

azinit commented Aug 30, 2022

@illright if u have another suggestions about scheme, u're welcome)

@illright
Copy link
Member

I think that we shouldn't have processes in that scheme. It's pretty vague and since we don't have any examples in our gallery that use processes, I imagine it would confuse people

@azinit
Copy link
Member

azinit commented Aug 31, 2022

@illright if u have another suggestions about scheme, u're welcome)

Also, if u have ideas about scheme styles and general UI, u can send figma/screens for help

@azinit
Copy link
Member

azinit commented Sep 6, 2022

@unordinarity Ну и поправить бы ошибку линтера и конфликты зарезолвить

@azinit azinit marked this pull request as draft April 24, 2023 09:53
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.

3 participants