-
Notifications
You must be signed in to change notification settings - Fork 29
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
updated thumbnail for wallpaper #303
Conversation
Demo starting at https://design-ubuntu-com-303.demos.haus |
templates/resources/index.html
Outdated
@@ -121,7 +121,7 @@ <h3 class="p-heading--5">Wallpapers</h3> | |||
class="col-4 u-hide--medium u-hide--small" | |||
> | |||
{{ image ( | |||
url="https://assets.ubuntu.com/v1/5d68f7aa-ubuntu-lunar-wallpaper-thumbnail.png", | |||
url="https://assets.ubuntu.com/v1/ec6cc515-wallpaper_minotaur_colour_1920x1080.png", | |||
alt="", | |||
width="1500", |
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.
@elioqoshi just to make sure image displays in correct aspect ratio please update the width and height props as well to be the same as image size (1920x1080).
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 aspect ratio should be the same, but I can update it as well.
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.
@bartaz updated the branch (sorry, I am a bit rusty with git and I think I didn't update the PR accordingly)
fixed thumbnail ratio
@bartaz is this good to go? |
I think so, I'm just trying to get the demo running just to QA it properly. |
updated the wallpaper thumbnail from Lunar to Mantic