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

Feature/Preprints Content Page - Mobile View [PREP-201] #224

Merged

Conversation

pattisdr
Copy link
Contributor

@pattisdr pattisdr commented Dec 7, 2016

Ticket

https://openscience.atlassian.net/browse/PREP-201

Purpose

Fixes several mobile-view issues on the preprints content page.

Changes

  • Edit button/logo were cut off on mobile view - removes logo entirely and moves edit button down to next row. Centers all text in header (title, contributors, dates, edit button).
    screen shot 2016-12-14 at 12 04 55 pm

  • "Download preprint" and version display were overtop of other data - moves to new line.
    screen shot 2016-12-14 at 12 05 44 pm

  • Distinguish "Primary" file differently in mobile view - different layout for supplementary file browser - mobile view

screen shot 2016-12-14 at 12 24 08 pm
screen shot 2016-12-14 at 12 25 18 pm

  • Change formatting of share icons - move to next line.
    screen shot 2016-12-14 at 12 05 49 pm

@pattisdr pattisdr changed the title Feature/Preprints Content Page - Mobile View Feature/Preprints Content Page - Mobile View [PREP-201] Dec 7, 2016
@pattisdr pattisdr force-pushed the feature/preprints-mobile-view branch from d259d7a to 21364d4 Compare December 13, 2016 20:51
@pattisdr
Copy link
Contributor Author

@icereval, so this is the PR you want me to upgrade ember-resolver dependency?

@brianjgeiger
Copy link
Contributor

I think it's just "whichever PR fixes the last places where it's wrong".

@icereval
Copy link
Contributor

@brianjgeiger is correct, if we can just wrap in @binoculars couple changes I think we'll be g2g and avoid the depreciation.

Copy link
Member

@binoculars binoculars left a comment

Choose a reason for hiding this comment

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

The mobile page looks much better now. No changes requested for that.

Thanks @pattisdr for updating the helpers to kebab-case. Let's keep the ember-resolver version pinned at 2.1.0 for now since it works with both kebab-case and camel-case. After yarn goes through, we can update it to the latest version (2.1.1 as of now) as the other packages). This is due to the fact that the yarn.lock file will need to be updated and committed. I'll put in another ticket just to update the dependencies and resolve any issues that occur as a result of the updates.

The ember-osf vesion is also causing a merge conflict, so just need to update that.

"ember-page-title": "3.0.6",
"ember-power-select": "1.0.0-beta.7",
"ember-read-more": "0.0.13",
"ember-resolver": "2.1.0",
"ember-resolver": "^2.0.3",
Copy link
Member

Choose a reason for hiding this comment

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

Let's leave this pinned at 2.1.0 for now, at least until yarn goes through, to avoid merge conflicts.

…er-preprints into feature/preprints-mobile-view

# Conflicts:
#	package.json
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