Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 7, 2025. It is now read-only.

Conversation

@jmancewicz
Copy link
Contributor

@jmancewicz jmancewicz commented Oct 12, 2016

Making the Model Store page more consistent with the DIGITS front page and adding a download progress bar.

Moving from this look;
old

To this look;
new

I also stopped the redirection to the front page after download was complete.

The progress bar is a progress line at the moment, under the model name to save real estate.

@jmancewicz jmancewicz force-pushed the modelstore-ui branch 3 times, most recently from 961af22 to 3a9f7a9 Compare October 13, 2016 01:58
@jmancewicz
Copy link
Contributor Author

@lukeyeager and @IsaacYangSLA maybe have a look.

<a href="/store/push?id={[model.id]}"> Import </a>
</td>
<td>
</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

I also have 'd' around here. However, the original HTML does not have that text.

Copy link
Contributor

@IsaacYangSLA IsaacYangSLA Oct 13, 2016

Choose a reason for hiding this comment

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

Yes. It's not there any more. It should be OK to merge.
@lukeyeager what do you think?

@IsaacYangSLA
Copy link
Contributor

I will merge it since it looks great and I didn't see anything broken.

@IsaacYangSLA IsaacYangSLA merged commit 9e977ed into NVIDIA:master Oct 13, 2016
SlipknotTN pushed a commit to cynnyx/DIGITS that referenced this pull request Mar 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants