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

Design System - Add containers #4370

Closed
pete-watters opened this issue Oct 18, 2023 · 14 comments · Fixed by #4655
Closed

Design System - Add containers #4370

pete-watters opened this issue Oct 18, 2023 · 14 comments · Fixed by #4655
Assignees
Labels
area:ui UI engineering specific tasks. effort:large Expected to take over 1 week of integration work enhancement

Comments

@pete-watters
Copy link
Contributor

pete-watters commented Oct 18, 2023

This issue is a new version of 4165 but with a more specific scope. The task is to implement a re-usable set of containers for page Header and Footer as per:

https://www.figma.com/file/2MLHeIeL6XPVi3Tc2DfFCr/%E2%9D%96-Leather-%E2%80%93-Design-System?type=design&node-id=5056-60851&mode=design&t=CbvgKq8c4jj6eRZE-0

272258460-6d12ef9e-c89a-4d1a-b2de-6ed02c829209 272258431-c40a35d3-a825-42a1-b2a6-5483bbb90492

The headers should be generic but will need to have some context provided as they will show different information in different views. e.g. in extension view, they need to show the network / account

This task relates to #4371 and builds upon #4165

@pete-watters
Copy link
Contributor Author

@markmhendrickson : we had a call yesterday to clear up the scope of #4165 . We have split up the work into two issues #4370 and #4371

CC @mica000 @kyranjamie @fabric-8

@markmhendrickson
Copy link
Collaborator

Thanks @pete-watters – can this particular issue be closed or is there more that remains for it specifically?

@pete-watters
Copy link
Contributor Author

pete-watters commented Oct 23, 2023

This one needs to be open as the work isn't done yet. This one is closed correctly

#4165

@pete-watters
Copy link
Contributor Author

As part of this task, consideration needs to be given to the colour of the header as per: #4253 (comment)

@markmhendrickson
Copy link
Collaborator

Related: #4615 (comment)

@pete-watters
Copy link
Contributor Author

Checking my new PR and setting the extension + popup width to 390 seems to resolve this issue

As we begin to scale up, past 390px width it starts to look buggy and weird again. I will see if I can improve it at the next breakpoint but I'm unsure how important that is if it looks OK in the standard popup size

Kapture.2023-12-20.at.12.02.40.mp4

@mica000
Copy link

mica000 commented Dec 20, 2023

@pete-watters Would it work to set the header width to 100% of the view port?

@pete-watters
Copy link
Contributor Author

@pete-watters Would it work to set the header width to 100% of the view port?

I can give it a try for the header, that will likely work. My point is do we want to have full responsiveness on all screens from sm 390px all the way to lg .

We could need to have an idea how the page responds at different widths. For example the header should stretch here but also the content width should extend too.

I'll get it working well in the main sizes and then see about the interim widths

@mica000
Copy link

mica000 commented Dec 20, 2023

My point is do we want to have full responsiveness on all screens from sm 390px all the way to lg .

Yes!!! that would be great: full responsiveness on all screens from sm 390px all the way to lg.

cc @fabric-8

@pete-watters
Copy link
Contributor Author

I've been progressing well on this:

  • making the home header consistent
  • improving the page headers

Yesterday @mica000 and I were speaking as for some pages with a two column layout we need to make some slight design tweaks to help us make things more consistent. That overlaps with #4423

To give more context, for the key entry / password screens:

  • most of the right column cards are fixed at 600px
  • the password / unlock screen has a fixed width of 500px
  • we don't have a specific header for those pages so we will work on that
  • the password session lock / unlock screen also needs to work on extension / popup mode and look well

@markmhendrickson markmhendrickson added this to the Establish UI library milestone Jan 2, 2024
@fabric-8
Copy link
Contributor

fabric-8 commented Jan 8, 2024

@pete-watters Michelly and I discussed header/footers the other day and would love to update the behaviour — Besides both being sticky, we'd also love to add a divider border to the header once content is scrolled — Here's a basic documentation of this change.

Please ping me if you have any questions or if there's the need for further discussion!

@markmhendrickson
Copy link
Collaborator

Note this work will fix the layout issues with the unlock screen during auth:

image

@pete-watters
Copy link
Contributor Author

@fabric-8, OK that's no problem. Better to do it now.

I had the footer being sticky so I will update it to also have a sticky header with a border showing for both on scroll.

@mica000
Copy link

mica000 commented Jan 9, 2024

Note this work will fix the layout issues with the unlock screen during auth:

image

@markmhendrickson Indeed! Here are the designs covering the onboarding flows: https://www.figma.com/file/2stUYm67q6Tkwe31CshODA/Onboarding?type=design&node-id=1%3A16709&mode=design&t=UX0gMMYcB50sCJHM-1

@kyranjamie kyranjamie added the effort:large Expected to take over 1 week of integration work label Jan 17, 2024
@markmhendrickson markmhendrickson linked a pull request Feb 20, 2024 that will close this issue
21 tasks
kyranjamie pushed a commit that referenced this issue Aug 8, 2024
## [6.45.0](v6.44.2...v6.45.0) (2024-08-08)

### Features

* add handle network item menu, closes [#5124](#5124) ([7a778a1](7a778a1))
* **analytics:** background analytics event ([967f7b1](967f7b1))
* implement mempool api, closes leather-io/issues[#215](#215) ([0608cb8](0608cb8))

### Bug Fixes

* account change in open frames ([ac21ff8](ac21ff8))
* default to first account as useCurrentStacksAccount returns undefined, ref [#5683](#5683) ([d67d238](d67d238))
* disable stx transfer if form is not valid ([8c9dc16](8c9dc16))
* empty logo in popup mode ([10af7db](10af7db))
* fix visual bug on hover of PSBT copy, ref [#4972](#4972) ([64d2fb6](64d2fb6))
* logo box show ([0cf3dae](0cf3dae))
* mainnet faker contract address ([17d71a8](17d71a8))
* remove sign out if there are no keys ([051b1b5](051b1b5))

### Internal

* force resolution of fast-xml-parser ([bbded02](bbded02))
* import DialogHeader from monorepo ([1878330](1878330))
* post-release merge back ([8a77da4](8a77da4))
* readd fragments to help escape single quote ([8ac4349](8ac4349))
* refactor containers to make them more composable, ref [#4370](#4370) ([231842f](231842f))
* refactor switch-account to use shared context ([10254d9](10254d9))
* refactor TwoColumn to remove unneeded fragments ([b5b7988](b5b7988))
* update org name ([da5cc05](da5cc05))
* update react-virtuoso package ([c874c00](c874c00))
* update ui and query packages ([5e2547c](5e2547c))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:ui UI engineering specific tasks. effort:large Expected to take over 1 week of integration work enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants