Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add additional feat-image controls for post headers #17

Merged
merged 14 commits into from
Feb 6, 2023

Conversation

jamesros161
Copy link
Collaborator

@jamesros161 jamesros161 commented Jan 23, 2023

Resolves #9

crio-2.19.0-issue-9.zip

Testing Instructions

  1. Install Crio and activate starter content ( or install an inspiration with a blog, then replace with this zip ).
  2. Open the Customizer and navigate to Design > Blog > Posts > Featured Images
  3. Ensure that you see four options for the Position control. They should be Header Background, Above Header, Below Header, and In Content
  4. Select all four options, ensuring they appear correctly in the preview, publish, and check that they appear correctly on the front end.
  5. When selecting Above Header and Below Header you will see another control appear called Featured Image Display Type which will allow you to select either Background or Image . Ensure both options display correctly in both positions, in the preview and when published.
  6. When choosing the type background you will see a slider show up allowing you to specify the height of the background. Ensure the control changes are reflected in the preview and when published, for both above and below header.
  7. When choosing the type image you can select the image size and alignment. Ensure these controls are working in the preview and when published.
  8. When choosing the position In Content ensure that the image size and alignment controls are shown, and work as expected.

@nicolepaschen

This comment was marked as resolved.

@nicolepaschen

This comment was marked as resolved.

@nicolepaschen

This comment was marked as resolved.

@jamesros161

This comment was marked as resolved.

@nicolepaschen

This comment was marked as resolved.

@nicolepaschen

This comment was marked as resolved.

@nicolepaschen
Copy link
Member

image

If I have Content Container for Page Titles set to full width and Featured Image Above Header / Image / Medium / Left the image appears right next to the page title

@jamesros161 jamesros161 merged commit d3c041e into branch-2.19.0 Feb 6, 2023
@jamesros161 jamesros161 deleted the issue-9 branch February 6, 2023 21:18
@jamesros161 jamesros161 mentioned this pull request Feb 6, 2023
@nicolepaschen

This comment was marked as resolved.

@nicolepaschen
Copy link
Member

% control doesn't work as expected, okay to just remove entirely.

jamesros161 added a commit that referenced this pull request Feb 14, 2023
* add filters for prev / next links (#22)

* add action for crio pro (#28)

* add action for crio pro

* undo margin-fix.scss change

* Crio pro issue 114 (#27)

* add action for crio pro

* add color-classes for links to comments

* add comment filters

* update edit exclusions

* update button edit-vars

* update _comments.scss

* set versions

* add show / hide date

* change selector for margin

* add min-height to heading w/ avatar

* add filters for prev / next links (#29)

* add Mega Menus to Pro Features (#8)

* add Mega Menus to Pro Features

* update versions for PR

* Add 'published' or 'modified' post meta date control (#13)

* set versions

* add control and filters for showing updated time

* Add alignment controls for post list 'entry-footer' links (#11)

* add blog page entry-footer link alignment controls

* fix off-center

* re-add missing comment icon controls

* Remove links from page / post / blog / archive titles (#15)

* remove link from page, post, blog & archive titles

* update versions for PR

* fix title color and title font size controls

* update versions

* Add additional feat-image controls for post headers (#17)

* update version for branch

* add templates for featured image positions

* update controls (#16)

* phpcs corrections

* update version for issue-9-alpha

* change bg height control

* add tooltip

* fix image padding / margins

* fix max-full-width flex direction

* MFW fixes

* fix 'hide' not working

* add js to place img inside content

* fix page title divided

* update versions

* update changelog

* minor css fixes

* fix hidden post image not showing on post list

* update versions for RC

* remove % units

* set version for stable

* add comments

* fix file dockblocks

* remove extra lines
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants