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

feat: add channels cards on mobile #368

Merged
merged 12 commits into from
Jul 31, 2024
Merged

Conversation

rolznz
Copy link
Contributor

@rolznz rolznz commented Jul 30, 2024

Moves the channel status calculation to the backend (returned in the API response), moves the channels table to a new component, adds a channels cards component, and extracts some shared code e.g. the ChannelWarning component

@rolznz rolznz requested a review from reneaaron July 30, 2024 14:37
@rolznz
Copy link
Contributor Author

rolznz commented Jul 31, 2024

@reneaaron merging for now, let me know if you have any comments

@rolznz rolznz merged commit 62181ab into fix/responsive Jul 31, 2024
8 checks passed
@rolznz rolznz deleted the feat/mobile-channel-cards branch July 31, 2024 05:27
rolznz added a commit that referenced this pull request Jul 31, 2024
* fix: scrollable sidebar

* fix: 2 rows of buttons on wallet page

* fix: try to make table scrollable

* feat: add channels cards on mobile (#368)

* feat: add channels cards on mobile

* feat: improve card design, reduce duplication

* fix: border rounding

* fix: badge styles

* fix: move link

* fix: move link to context menu

* fix: add positive colors to other themes

* fix: logo clipPath

* fix: introduce warning color + badge

* fix: nesting warning

---------

Co-authored-by: René Aaron <rene@twentyuno.net>

* fix: app header content alignment

---------

Co-authored-by: Roland Bewick <roland.bewick@gmail.com>
Co-authored-by: Roland <33993199+rolznz@users.noreply.github.com>
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