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

My table layout and collection info #567

Merged
merged 1 commit into from
Jul 3, 2014
Merged

My table layout and collection info #567

merged 1 commit into from
Jul 3, 2014

Conversation

awead
Copy link
Contributor

@awead awead commented Jul 2, 2014

This adds additional information in the my file listing that displays what collections a file is a part of. There's also some cosmetic changes in the tables, such as not displaying the batch edit toggle in the collection view, and using chevrons instead of plus/minus signs. This makes the UI more consistent and
☀️ 🌈 ☀️ 🌈 ☀️ 🌈 ☀️ 🌈 ☀️ 🌈 ☀️ 🌈 ☀️ 🌈 ☀️ 🌈 ☀️ 🌈 ☀️ 🌈 ☀️ 🌈 ☀️ 🌈

@@ -98,14 +98,14 @@ Blacklight.onLoad(function() {

// show/hide more information on the dashboard when clicking
// plus/minus
$('.glyphicon-plus').on('click', function() {
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd really like this moved out of sufia.js into it's own file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed.

@awead
Copy link
Contributor Author

awead commented Jul 2, 2014

@jcoyne this has been updated with helper tests, instead of the feature tests. I also change a couple of sass files to use shared variables, and move the request js into another file.

jcoyne added a commit that referenced this pull request Jul 3, 2014
My table layout and collection info
@jcoyne jcoyne merged commit bda2297 into master Jul 3, 2014
@jcoyne jcoyne deleted the file-table branch July 3, 2014 01:24
jcoyne pushed a commit that referenced this pull request Nov 17, 2016
Restore additional_response_formats for FileSets
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.

4 participants