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

Gallery dataset redesign #2761

Merged
merged 23 commits into from
Jun 24, 2018
Merged

Gallery dataset redesign #2761

merged 23 commits into from
Jun 24, 2018

Conversation

MichaelBuessemeyer
Copy link
Contributor

@MichaelBuessemeyer MichaelBuessemeyer commented Jun 18, 2018

Mailable description of changes:

  • The dataset gallery got a redesign with mobile support.

URL of deployed dev instance (used for testing):

Steps to test:

  • open the dataset gallery
  • resize the window and watch the dataset resize according to the windows width

Issues:


  • Ready for review

Copy link
Member

@philippotto philippotto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The responsive design works very well 👍 Unfortunately, the right hand side of the cards does not render properly for me (chrome & firefox). It looks like this for me:

image

The horizontal scrollbar shouldn't be there and the text is a bit scrambled. Does it work for your computer @MichaelBuessemeyer ? Maybe you forgot to push some changes?

Also, you need to update the snapshots since you changed the dashboard. You can read this wiki section for details. Just ask me if you have questions! However, you can wait with this until everything else is resolved :)

@@ -158,3 +162,7 @@ const mapStateToProps = (state: OxalisState): StateProps => ({
});

export default connect(mapStateToProps)(GalleryDatasetView);

// TODO: mit mediaquery bei kleinen Bildschirmen: display: block; und breite beider spans auf 100% setzen !!!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's up with this todo? :) Still relevant?

background: rgba(0, 0, 0, 0.8);
display: inline-block;
.spotlight-item-card {
height: 300px;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the height 300px when we request a height of 400px from the server? Shouldn't we request 300px then?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because if the width is below 600px the thumbnails is displayed in full width. If you subtract padding and all you get a max width of 512px that is about 500px.

}
}

.dataset-description-body{ //TODO set the hight of the body correct and let it appear below the heading/title
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the todo resolved?



.dataset-thumbnail {
height: 100%;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The indentation is a bit off here. If you use sublime, the package JsPrettier should also format less files for you if you want to save yourself some trouble :)

Copy link
Member

@philippotto philippotto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, works well for me 👍

@MichaelBuessemeyer MichaelBuessemeyer merged commit bf685b3 into master Jun 24, 2018
@MichaelBuessemeyer MichaelBuessemeyer deleted the gallery-dataset-redesign branch June 24, 2018 10:30
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.

Redesign Dataset Gallery
2 participants