status-mobile
status-mobile copied to clipboard
Quo2 components - Drawers / Drawer Top
Implement this component in the quo2 component library as per the design spec:
Designs - Drawers / Drawer Top https://www.figma.com/file/WQZcp6S0EnzxdTL4taoKDv/Design-System-for-Mobile?node-id=2500%3A42270&t=29WaMjPTYMtHqa1n-0
** There is an implementation of one variation of this more or less done in https://github.com/status-im/status-mobile/blob/develop/src/status_im2/contexts/communities/menus/request_to_join/view.cljs#L83-L98
Acceptance Criteria
dark and light mode component tests (jest) quo2 preview export from quo2/core.cljs adjust usage in communities join screen to use this component
@smohamedjavid , @ulisesmac @ajayesivan any of you know if this is a duplicate now? do we need it anymore??
At first glance, #17021 is a duplicate of this and it looks like there are a few new types added in the Figma design compared to the screenshot in the issue.
I'm working on an issue related to this component for the community account selection feature. During that, I will also verify this.
Hi @J-Son89,
I noticed that the type
prop of this component has a variety of values, including combinations like "Default + Description," which strikes me as somewhat unconventional. It seems to blur the distinction between different types of information. In my opinion, it might be clearer to have separate props for each distinct type of information.
What do you think, should we talk to the designers to clarify this, or is this a pattern that we use with other components as well?
Types
- Default
- Default + Description
- Context tag
- Context tag + button
- Info
- Info + Description
- Account
- Label
- Keypair
- Keypair
- Keycard
- Default keypair
- Counter
- Token
- Token hidden
- 2 Context
- 2 Context + button
Hey @ajayesivan, of course. I think that's fine to use your best judgement. I think ideally design would just change this component to improve the developement experience and make it easier to implement, in that I still think it is worth discussing them the pain points here, although it might not be possible for them to change it here without making adjustments to many design files?? 🤔
In any case we really need to revisit how the designers are building the design system because it doesn't scale well with the development process. @ilmotta has discussed this in a few prs and I have requested that 3/4 developers talk with designers in the offsite and bring some feedback of what can be improved in the design system approach as it is something we really need to get on top of 👍
Hey @ajayesivan, of course. I think that's fine to use your best judgement. I think ideally design would just change this component to improve the developement experience and make it easier to implement, in that I still think it is worth discussing them the pain points here, although it might not be possible for them to change it here without making adjustments to many design files?? 🤔
In any case we really need to revisit how the designers are building the design system because it doesn't scale well with the development process. @ilmotta has discussed this in a few prs and I have requested that 3/4 developers talk with designers in the offsite and bring some feedback of what can be improved in the design system approach as it is something we really need to get on top of 👍
Thank you very much @J-Son89 for using the offsite time to bring this very important topic to designers 😍