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

CI adjustment - Pinkify Legend #1766

Merged
merged 3 commits into from Jul 5, 2023
Merged

CI adjustment - Pinkify Legend #1766

merged 3 commits into from Jul 5, 2023

Conversation

arbadacarbaYK
Copy link
Contributor

To harmonize with the Shop also Legend should become pink - and so the flamingo theme, favicon and images are adjusted.

Images are linked on https://cdn.jsdelivr.net/gh/lnbits/lnbits@0.3.0/docs/logos/ (core/views/generic.py)
so this needs an update as well.
Backup of new files can be found on https://drive.google.com/drive/folders/1Vjbr8lfXjNV2IKhWfrjEmbwxjZGJwb_4?usp=sharing

@codecov-commenter
Copy link

codecov-commenter commented Jun 16, 2023

Codecov Report

Merging #1766 (1e02f13) into main (e13426a) will not change coverage.
The diff coverage is n/a.

❗ Current head 1e02f13 differs from pull request most recent head 7d92313. Consider uploading reports for the commit 7d92313 to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main    #1766   +/-   ##
=======================================
  Coverage   55.47%   55.47%           
=======================================
  Files          47       47           
  Lines        7264     7264           
=======================================
  Hits         4030     4030           
  Misses       3234     3234           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@blackcoffeexbt
Copy link
Collaborator

@arbadacarbaYK Thanks :) Changes look good to me! @dni Can this be merged pls?

Add files via upload
Update base.scss

background was overlapping on invert 😬
package.json Outdated Show resolved Hide resolved
@dni dni requested review from motorina0, arcbtc and callebtc July 4, 2023 20:00
fix flamingo

bundle
@motorina0
Copy link
Collaborator

LGTM, a screenshot would be nice!

@dni dni merged commit 83a04d8 into lnbits:main Jul 5, 2023
13 checks passed
@arbadacarbaYK
Copy link
Contributor Author

LGTM, a screenshot would be nice!

its more about the logo than the theme. All pages are mixed nowadays in violet/pink, trying to reduce that

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.

None yet

6 participants