Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dni
Copy link
Member

@dni dni commented Jun 21, 2024

simplified and improved responsiveness

screenshot-1718971749
screenshot-1718971302
screenshot-1718971295
screenshot-1718971288

give it more luft and simplify it a bit. also improved responsiveness
@codecov
Copy link

codecov bot commented Jun 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.00%. Comparing base (e9d6160) to head (bd6b0fe).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2558      +/-   ##
==========================================
- Coverage   61.24%   61.00%   -0.25%     
==========================================
  Files          69       69              
  Lines        9914     9914              
==========================================
- Hits         6072     6048      -24     
- Misses       3842     3866      +24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@motorina0 motorina0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@arcbtc
Copy link
Member

arcbtc commented Jun 24, 2024

I think its nice to keep the short description once activated, space is not an issue, as most people will only have a dozen or so extensions enabled, and the "new version" badge seems a bit floaty.
Original looks cleaner to me:
image

@blackcoffeexbt
Copy link
Collaborator

@dni I'd agree with @arcbtc on this. Keeping the short desc visible is important and the "new version" label should be positioned somewhere that it feels more "docked" to the card. Closer to a corner with a fixed distance would be good.

@arcbtc
Copy link
Member

arcbtc commented Jun 25, 2024

I like @dni chnages, looks waaaayyy better on mobile, but I think we should keep the short description

@dni dni requested a review from arcbtc June 26, 2024 11:01
@dni dni merged commit cbe0861 into dev Jun 26, 2024
@dni dni deleted the extpage branch June 26, 2024 11:07
ram0verflow pushed a commit to ram0verflow/lnbits that referenced this pull request Jul 3, 2024
* feat: improve on extension page layout
give it more luft and simplify it a bit. also improved responsiveness
* show description now, ben
motorina0 pushed a commit that referenced this pull request Jul 10, 2024
* feat: improve on extension page layout
give it more luft and simplify it a bit. also improved responsiveness
* show description now, ben
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.

4 participants