Skip to content

INS-3604 Add new logo and update version#61

Merged
DanielCoelhoOliveira97 merged 8 commits intomasterfrom
INS-3604-update-logo-on-word-press-plugin-page-to-reflect-new-siteimprove-branding
Aug 28, 2025
Merged

INS-3604 Add new logo and update version#61
DanielCoelhoOliveira97 merged 8 commits intomasterfrom
INS-3604-update-logo-on-word-press-plugin-page-to-reflect-new-siteimprove-branding

Conversation

@DanielCoelhoOliveira97
Copy link
Contributor

No description provided.

@mostergaard
Copy link
Contributor

@DanielCoelhoOliveira97 could you post some screenshots here on how it looks in WP where the new logo is updated? I did notice that the updated icon is WAY higher resolution, so I wonder if there could be some scaling issues.

@DanielCoelhoOliveira97
Copy link
Contributor Author

@DanielCoelhoOliveira97 could you post some screenshots here on how it looks in WP where the new logo is updated? I did notice that the updated icon is WAY higher resolution, so I wonder if there could be some scaling issues.

@mostergaard I managed to setup the wordpress env locally to test the changes, and you were right (it was breaking the layout), so I changed the dimensions to be the same as before

@rdom-si
Copy link
Contributor

rdom-si commented Aug 26, 2025

@DanielCoelhoOliveira97 I just noticed there is, in class-siteimprove-admin.php, a hardcoded svg string for the Siteimprove logo on the prepublish button. Either that needs to be updated to the correct svg string, or it would be nice to have the code "import" the svg from the actual file (this would be my recommendation)

@DanielCoelhoOliveira97 DanielCoelhoOliveira97 force-pushed the INS-3604-update-logo-on-word-press-plugin-page-to-reflect-new-siteimprove-branding branch from 95ab615 to 421161d Compare August 27, 2025 14:06
@DanielCoelhoOliveira97 DanielCoelhoOliveira97 merged commit 8d2727f into master Aug 28, 2025
1 check passed
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