Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

[OYPD-182] Full width image for subcategory list below 480px #137

Merged
merged 2 commits into from Feb 14, 2017

Conversation

david-hernandez
Copy link
Contributor

@david-hernandez david-hernandez commented Feb 10, 2017

  • Create a program page that can list subprograms using the program list paragraph type.
  • Look at the program list on smaller screen sizes.
  • See the image is full width on screens below 480px.

We may want to center the image between 480 and 760, but I wouldnt make the image max width. It will expand a 270px image to fit 700+ width which will look bad.

screen shot 2017-02-10 at 12 34 17 pm

@Sanchiz
Copy link
Contributor

Sanchiz commented Feb 13, 2017

@david-hernandez let's center image on screen 480px-768px without max-width.

@Sanchiz Sanchiz added PR: Code Reviewed Code reviewed. Ready for QA PR: Needs Work Unfinished task. Issues still there labels Feb 13, 2017
@gundevel
Copy link
Collaborator

Build comment file:

===============
Build site installed at http://openy-dev.ffwua.com/build443


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy-dev.ffwua.com:8080/job/PR_BUILDER/443/

Copy link
Contributor

@Sanchiz Sanchiz left a comment

Choose a reason for hiding this comment

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

Approved.

@gundevel
Copy link
Collaborator

@Sanchiz Sanchiz merged commit 94b6eb2 into ymcatwincities:8.x-1.x Feb 14, 2017
hamrant pushed a commit to hamrant/openy that referenced this pull request Feb 16, 2017
[OYPD-182] Full width image for subcategory list below 480px
@david-hernandez david-hernandez deleted the OYPD-182 branch July 20, 2017 15:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: Code Reviewed Code reviewed. Ready for QA PR: Needs Work Unfinished task. Issues still there
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants