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

Related Posts widget: make byline and thumbnail placement options work #1243

Merged
merged 5 commits into from
Jun 28, 2016

Conversation

benlk
Copy link
Collaborator

@benlk benlk commented Jun 27, 2016

Changes

  • "show date" option text is renamed to "show byline" to match internal variable
  • show_byline option now actually toggles the presence of the byline; defaults to not
  • Before/After headline option for thumbnail placement now has effect
  • Styles for proper alignment of thumbnail in before and after positions

Why

Because we had these options in the widget, but they weren't doing anything.

For #1242

@benlk benlk added the priority: high Either blocks work on a priority-normal task or a solution here informs other work. label Jun 27, 2016
@benlk benlk added this to the hotfix milestone Jun 27, 2016
@aschweigert
Copy link

Is the before/after option for the thumbnail a thing we actually want/need? Seems overly complicated for me unless there's a particularly good reason for it.

@benlk
Copy link
Collaborator Author

benlk commented Jun 27, 2016

I'm in favor of removing it. It is a needless complication, and it makes more sense for that to be a child-theme styles decision than a per-widget decision.

@aschweigert
Copy link

go for it

@benlk
Copy link
Collaborator Author

benlk commented Jun 27, 2016

D we want an option to disable showing the thumbnail at all?

…Related Posts widget, and clean up styles to make it easier for child themes to modify in the future
@benlk
Copy link
Collaborator Author

benlk commented Jun 27, 2016

Before/After option and styles have been removed.

@aschweigert
Copy link

the markup that uses that option is still in the widget though...right?

@benlk
Copy link
Collaborator Author

benlk commented Jun 27, 2016

Not sure what you mean by "the markup that uses that option" - the only difference to the markup on the thumbnail and the link inside it is that the thumbnail no longer has an alignleft class on it, and instead gets the float: left from the widget's styles.

@aschweigert
Copy link

I see, there was just a comment in there that made it look like that option was still present. Removed that and I think we're fine to merge this.

@aschweigert aschweigert merged commit 13fb906 into master Jun 28, 2016
@aschweigert aschweigert deleted the 1242-related-posts-byline-checkbox branch June 28, 2016 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high Either blocks work on a priority-normal task or a solution here informs other work.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants