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

fix: add favicon to extension pages (firefox) #1857

Merged
merged 1 commit into from
Dec 12, 2022
Merged

fix: add favicon to extension pages (firefox) #1857

merged 1 commit into from
Dec 12, 2022

Conversation

im-adithya
Copy link
Member

Describe the changes you have made in this PR

Favicons in extension pages

Link this PR to an issue [optional]

--

Type of change

(Remove other not matching type)

  • fix: Bug fix (non-breaking change which fixes an issue)

Screenshots of the changes [optional]

Screenshot 2022-12-12 at 5 15 40 PM

How has this been tested?

Manually

Checklist

  • My code follows the style guidelines of this project and performed a self-review of my own code
  • New and existing tests pass locally with my changes
  • I checked if I need to make corresponding changes to the documentation (and made those changes if needed)

@github-actions
Copy link

🚀 Thanks for the pull request!

Here are the current build files for testing:

Download and unzip the file for your browser. Refer to the readme for detailed install instructions.


This build is brought to you by: Adam Fiscor (who recently dropped 1337 sats):

The future is bright. We just have a lot of work to do.

Want to sponsor the next build? send some sats to ⚡️builds@getalby.com (don't forget to provide your name)

Don't forget: keep earning sats!

Copy link
Contributor

@escapedcat escapedcat left a comment

Choose a reason for hiding this comment

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

ack

@jankoegel
Copy link
Contributor

@im-adithya looks good. should we add the same to popup.html and prompt.html!?
popup.html probably never appears in a tab of its own!?!
I don't know when prompt.html is rendered.

@escapedcat
Copy link
Contributor

popup.html probably never appears in a tab of its own!?!

Right, I don't think so.

I don't know when prompt.html is rendered.

This is being prompted if you interact with a website i.e. "connect", "do a payment", etc.
Does never show in a tab afaik

@@ -4,6 +4,7 @@
<head>
<meta charset="UTF-8" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<link rel="icon" type="image/png" href="assets/icons/alby_icon_yellow_32x32.png">
Copy link
Collaborator

Choose a reason for hiding this comment

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

@escapedcat this path might be relevant for v3. - there was some change in the paths, wasn't it?

Copy link
Contributor

Choose a reason for hiding this comment

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

It should not matter for the metatags but will check

@bumi bumi merged commit b58b2b9 into master Dec 12, 2022
@bumi bumi deleted the task-favicon-ff branch December 12, 2022 15:45
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