starred/pocket-id
1
0
Fork 0
mirror of https://github.com/pocket-id/pocket-id.git synced 2025-12-06 09:13:19 +03:00
Code Issues 121 Packages Projects Releases 16 Wiki Activity

🐛 Bug Report: PocketID icon in mails is too large #296

New Issue
Closed
opened 2025-10-07 00:09:17 +03:00 by OVERLORD · 9 comments
OVERLORD commented 2025-10-07 00:09:17 +03:00
Owner
Copy Link

Originally created by @alexlehm on GitHub.

Reproduction steps

When I look at the pocket-id mails, the P logo takes up the whole page, likely because the SVG is rendered to fit the page width

Expected behavior

The image should be maybe 100x100 pixel

Actual Behavior

Image

Version and Environment

pocket 0.40.0

the webmail is called SnappyMail

Log Output

No response

Originally created by @alexlehm on GitHub. ### Reproduction steps When I look at the pocket-id mails, the P logo takes up the whole page, likely because the SVG is rendered to fit the page width ### Expected behavior The image should be maybe 100x100 pixel ### Actual Behavior ![Image](https://github.com/user-attachments/assets/b4053554-4570-4532-9e4b-5ccfc0fe70eb) ### Version and Environment pocket 0.40.0 the webmail is called SnappyMail ### Log Output _No response_
OVERLORD added the bug label 2025-10-07 00:09:17 +03:00
OVERLORD commented 2025-10-07 00:09:20 +03:00
Author
Owner
Copy Link

@stonith404 commented on GitHub:

How does the rest of the mail look like?

@stonith404 commented on GitHub: How does the rest of the mail look like?
OVERLORD commented 2025-10-07 00:09:20 +03:00
Author
Owner
Copy Link

@kmendell commented on GitHub:

the style for the header logo is set in the style_html.tmpl:

 .header .logo img {
     width: 32px;
     height: 32px;
     vertical-align: middle;
}

My guess is some email cleints are disreagrding these.

@alexlehm Can you try this image to see if its fixed? ghcr.io/kmendell/pocket-id:email-icon

@kmendell commented on GitHub: the style for the header logo is set in the style_html.tmpl: ``` .header .logo img { width: 32px; height: 32px; vertical-align: middle; } ``` My guess is some email cleints are disreagrding these. @alexlehm Can you try this image to see if its fixed? `ghcr.io/kmendell/pocket-id:email-icon`
OVERLORD commented 2025-10-07 00:09:21 +03:00
Author
Owner
Copy Link

@alexlehm commented on GitHub:

this can probably be changed by giving an explicit size for the image

  <img src="{{ .LogoURL }}" width="100" height="100" alt="{{ .AppName }}"/>

in backend/resources/email-templates/login-with-new-device_html.tmpl

@alexlehm commented on GitHub: this can probably be changed by giving an explicit size for the image <img src="{{ .LogoURL }}" width="100" height="100" alt="{{ .AppName }}"/> in backend/resources/email-templates/login-with-new-device_html.tmpl
OVERLORD commented 2025-10-07 00:09:21 +03:00
Author
Owner
Copy Link

@paul933 commented on GitHub:

Same on Outlook app

Image (100% zoom)

Image(50% zoom)

@paul933 commented on GitHub: Same on Outlook app ![Image](https://github.com/user-attachments/assets/8b883aef-f49f-4a91-9c1d-f62dfb9ad253) (100% zoom) ![Image](https://github.com/user-attachments/assets/b911b178-24a0-4af6-938c-aed5a3bdbbbf)(50% zoom)
OVERLORD commented 2025-10-07 00:09:22 +03:00
Author
Owner
Copy Link

@alexlehm commented on GitHub:

the rest looks fine

Image

@alexlehm commented on GitHub: the rest looks fine ![Image](https://github.com/user-attachments/assets/a27a27bd-f0b1-493f-810f-6499a85765cf)
OVERLORD commented 2025-10-07 00:09:23 +03:00
Author
Owner
Copy Link

@paul933 commented on GitHub:

Image

Fix for me in Outlook 👍

@paul933 commented on GitHub: ![Image](https://github.com/user-attachments/assets/6ad887ed-3cd4-4dd9-8819-04bd2a3cd405) Fix for me in Outlook 👍
OVERLORD commented 2025-10-07 00:09:23 +03:00
Author
Owner
Copy Link

@kmendell commented on GitHub:

@stonith404 i pushed my fix to the branch fix/email-icon , im noticing differences with the screenshots in this issue vs how mine look, my logo is aligned to the left of the header but these screenshots are not.

Not sure if you wanna take a look at that part. Of if thats also a side effect of email client css rendering.

@kmendell commented on GitHub: @stonith404 i pushed my fix to the branch `fix/email-icon` , im noticing differences with the screenshots in this issue vs how mine look, my logo is aligned to the left of the header but these screenshots are not. Not sure if you wanna take a look at that part. Of if thats also a side effect of email client css rendering.
OVERLORD commented 2025-10-07 00:09:23 +03:00
Author
Owner
Copy Link

@stonith404 commented on GitHub:

The email still looks strange. It should look much better in other clients like Gmail or Apple Mail but I guess your clients just support css very limited.

@stonith404 commented on GitHub: The email still looks strange. It should look much better in other clients like Gmail or Apple Mail but I guess your clients just support css very limited.
OVERLORD commented 2025-10-07 00:09:24 +03:00
Author
Owner
Copy Link

@alexlehm commented on GitHub:

that looks good in my mail page

Image

@alexlehm commented on GitHub: that looks good in my mail page ![Image](https://github.com/user-attachments/assets/e973d747-0868-48af-9987-9755c0f46e35)
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
breaking
bug
documentation
feature
needs more upvotes
open to pull requests
pull-request
Mirrored from GitHub Pull Request
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id#296
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?