-
Notifications
You must be signed in to change notification settings - Fork 144
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
convert dashboard-pages to tailwind #540
Conversation
@kayceeDev sync with main please |
I will definitely do that in my next push. |
2bd4266
to
63c64f5
Compare
@whateverfw The main is also broken. but I will try and fix them. |
what main? do you mean that main branch also has issue on this pages? It can be the case, because we are in the process of tailwind migration, so maybe someone somehow broke those pages. That's basically why we have this ticket to fix exactly those pages and another ones are fixed by another devs. |
Yes, I will definitely fix them. In the mobile, I see a drop down menu popping out. I think the active state changes to true on mobile.. |
if that's blocking you and it is not a quick fix, just put some false state for now and concentrate on your pages, we'll fix it separately |
Alright. Thanks |
So the broken you are referring to for mobile and desktop, is it during the break points? |
@kayceeDev I just attached all the screenshot, in general on few pages on desktop only cards styles are broken, however position history page has broken both desktop in mobile and in general each page's mobile is not correct, as you can see on screenshots. I don't see any breakpoints for responsiveness in code |
I will fix it. Thank you for the screenshots. I didn't clearly see the mobile completely cos of the overlay that comes up whenever you go to mobile. |
bbe674a
to
4ffaebf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see that mobile styles are not yet ideal, but better, pls take a look carefully and fix.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please revert changes on in this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright. I will. Remove it from my commit
87686c1
to
1aa3ff4
Compare
1aa3ff4
to
29e62c4
Compare
@whateverfw all fixes has been resolved and everything looks good on all devices now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make sure to go through figma and make sure those pages looks like that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still appears like that @kayceeDev
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still do not match figma desktop/mobile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of the components on mobile has extra buttons |
Please clarify. Because some of the differences I see on mobile is the cards I turned to column instead of rows |
1 similar comment
Please clarify. Because some of the differences I see on mobile is the cards I turned to column instead of rows |
make existing, if it didn't exist before - do not add |
yeah, so, that was exactly one of the points, which aren't like in Figma |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still various broken styles or different looks from figma/previous version.
9e581de
to
cdaa6fa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kayceeDev
I see that direction for fixes of position history is incorrect, please follow the figma, we had 3 columns and then action button that triggers a modal with the rest of the fields...
ee56bd7
to
a1ad7ea
Compare
@whateverfw fixed |
18aa6a5
to
8254b9e
Compare
8254b9e
to
016bd08
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
No description provided.