Skip to content
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

Thumbnail #931

Merged
merged 3 commits into from May 26, 2016
Merged

Thumbnail #931

merged 3 commits into from May 26, 2016

Conversation

fe-kre
Copy link
Contributor

@fe-kre fe-kre commented May 17, 2016

I am sorry again for my approach, I am really new to Github and learn their procedures.

I correct my mistake at #922.

Hopefully, now everything works as expected.

Add Option for .preview-thumb
Create preview-thumb as a 'div-class'
Changed previev -> preview
@fe-kre
Copy link
Contributor Author

fe-kre commented May 25, 2016

Is there any problems with my PR? Shall I correct something?

@rsinghal
Copy link
Collaborator

I haven't had a chance to look at it yet, but plan to today.

@rsinghal
Copy link
Collaborator

@fe-kre It looks fine. In general, I'm not sure what the best behavior is when the user resizes the window down. The default behavior is to have the ... display when it is not possible to display a full thumbnail, but there is an argument to be made that it should follow the behavior of your large scroll - always show a little bit of the first thumbnail. I am going to ticket that separately and merge this.

@rsinghal rsinghal merged commit 6cf7f91 into ProjectMirador:release2.1 May 26, 2016
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.

None yet

2 participants