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: dropdown changes #4668

Merged
merged 12 commits into from
Mar 28, 2024
Merged

style: dropdown changes #4668

merged 12 commits into from
Mar 28, 2024

Conversation

mstrasinskis
Copy link
Contributor

@mstrasinskis mstrasinskis commented Mar 27, 2024

Motivation

Update style for project selector on mobile to provide more space for the content. A first step to the new split content design. Uses changes of gix-component pr - dfinity/gix-components#403

Changes

  • npm run update:gix
  • add noMarginBottom to remove bottom margins. To avoid stretching of a SplitContent start block on mobile.
  • redefine some SplitContent css values.

Tests

Manually.

Todos

  • Add entry to changelog (if necessary).

Screenshots

Before After
image image

|

@mstrasinskis mstrasinskis marked this pull request as ready for review March 27, 2024 16:51
@mstrasinskis mstrasinskis requested review from a team as code owners March 27, 2024 16:51
@mstrasinskis mstrasinskis requested a review from dskloetd March 27, 2024 16:52
@mstrasinskis mstrasinskis marked this pull request as draft March 27, 2024 16:52
@mstrasinskis mstrasinskis marked this pull request as ready for review March 28, 2024 08:38
dskloetd
dskloetd previously approved these changes Mar 28, 2024
mstrasinskis added a commit to dfinity/gix-components that referenced this pull request Mar 28, 2024
# Motivation

`gix-components` changes for mobile project selector redesign. Part of
dfinity/nns-dapp#4668

# Changes

- `noMargin ` prop to remove margins from a `Card`. To avoid stretching
of a `SplitContent` start block on mobile.
- `--nav-component-title-display` css-variable to control the nav title
to control it's visibility from outside.
- don't display `SplitContent` start background on mobile.

# Screenshots
| Before | After |
|--------|--------|
| <img width="326" alt="image"
src="https://github.com/dfinity/gix-components/assets/98811342/bf644db9-ae48-4ab1-a429-500ac6fa2f96">
| <img width="310" alt="image"
src="https://github.com/dfinity/gix-components/assets/98811342/46c6e7ef-d428-46de-ac4a-3b932f8a3819">
|
 |

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
@mstrasinskis mstrasinskis added this pull request to the merge queue Mar 28, 2024
@mstrasinskis mstrasinskis removed this pull request from the merge queue due to a manual request Mar 28, 2024
@mstrasinskis mstrasinskis added this pull request to the merge queue Mar 28, 2024
Merged via the queue into main with commit 6cd668a Mar 28, 2024
50 checks passed
@mstrasinskis mstrasinskis deleted the project-dropdown-style-adjustments branch March 28, 2024 16:34
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.

2 participants