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

Skip to content

Conversation

@arcbtc
Copy link
Member

@arcbtc arcbtc commented Jun 25, 2024

Fixes carousel and adv desc dialog. Extensions page also had a div in the wrong place

Carousel was stretching images
After fix:
Screenshot from 2024-06-25 19-15-41

Before:
Screenshot from 2024-06-25 19-14-53

Adv desc looking heaps better on mobile

video_2024-06-25_21-55-45.mp4

@codecov
Copy link

codecov bot commented Jun 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.25%. Comparing base (cbe0861) to head (9b6fc6a).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2562   +/-   ##
=======================================
  Coverage   61.25%   61.25%           
=======================================
  Files          69       69           
  Lines        9914     9914           
=======================================
  Hits         6073     6073           
  Misses       3841     3841           

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

@arcbtc arcbtc changed the title Carousel image fix Carousel and adv desc fix Jun 25, 2024
@motorina0 motorina0 requested review from dni and talvasconcelos June 26, 2024 06:07
@dni dni changed the title Carousel and adv desc fix bug: carousel and adv desc fix Jun 26, 2024
@dni dni merged commit 7f62894 into dev Jun 26, 2024
@dni dni deleted the fixcarousel branch June 26, 2024 11:21
ram0verflow pushed a commit to ram0verflow/lnbits that referenced this pull request Jul 3, 2024
* Carousel image fix
* pushed carousel nav to top as well so yt controls are accessible
* Added some extra breathing room for the adv description
motorina0 pushed a commit that referenced this pull request Jul 10, 2024
* Carousel image fix
* pushed carousel nav to top as well so yt controls are accessible
* Added some extra breathing room for the adv description
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.

3 participants