-
-
Notifications
You must be signed in to change notification settings - Fork 43
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
Update Lutra logo and description #565
base: main
Are you sure you want to change the base?
Conversation
Thanks @harminius - @Xpirix please confirm if the image size is ok... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @harminius
@timlinux
For the changes in content/funders
, I would suggest editing the information at https://changelog.qgis.org/en/qgis/member/ instead. The files inside this folder will be automatically overridden by the information from https://changelog.qgis.org/en/qgis/members/json/. The logo will be resized automatically as well.
The logo size for static/img/contributors/lutra_consulting.png
looks good to me. The file size is small (11.3 KB) even if the scale is wider.
However, the width will not be fully shown at https://qgis.org/resources/support/commercial-support/ as the displayed size there is only 147px. If possible, I would suggest keeping the width similar to the old one to be consistent with the displayed size. But both cases are okay.
@harminius and @Xpirix - I will update the logo in the changelog application (for the sustaining membership listing) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The logo content/funders/www.lutraconsulting.co.uk.png
has already been updated from the changelog. I think it doesn't need to be updated anymore.
Afterwards, it looks good to me. Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This logo has already been updated from the changelog. Please see:
https://github.com/qgis/QGIS-Website/blob/main/content/funders/www.lutraconsulting.co.uk.png
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Xpirix, I reverted the image change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
With the facelift of Lutra's website a new logo comes to life:
🦦
Let the QGIS website reflect that.