Conversation
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.
Looks much better / Thank you!
What do you think about the code duplication?
if (isSearching) { | ||
this._setSize(slice, Math.min(this.maxHeight, h)); | ||
break; | ||
} |
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.
continue w
is replaced by isSearching
: try to make the code more readable.
.result-images a:focus .img-thumbnail { | ||
box-shadow: 5px 5px 15px 0px black; | ||
} | ||
.result-images.js a .img-thumbnail { |
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.
ImageLayout
adds the js
class for each processed image.
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.
After a rebase it is good to go.
Adjust thumbnail sizes to fill the container width
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.
Can this be merged soon? I've had this PR merged on my fork for a little over a week now and it works great!
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.
Great, thanks!
What does this PR do?
With javascript
Without javascript
Why is this change important?
Improve UX.
How to test this PR locally?
use the different oscar styles, search for
!images time
!general !images time
Try javascript enable and disabled.
Author's checklist
Related issues