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

fix: resolve issue with byline appearing inline with resource title on home feed #258

Merged
merged 4 commits into from
Feb 21, 2020

Conversation

jhung
Copy link
Contributor

@jhung jhung commented Feb 21, 2020

Description

Fixes #257 .

Steps to test

  1. Go to Layouts > Home
  2. Notice how byline is on a separate line from the title.
  3. Edit home.config.js lines 63 and 64 to generate 1 word title and bylines (i.e. min: 1, max: 1).
  4. Rebuild pinecone with this new configuration npm run build
  5. Run fractal start
  6. Go to Layouts > Home
  7. Notice that bylines are still on separate lines even though the titles and bylines are short.

Expected behavior:
The byline for a resource should be on its own line regardless of the length of the title or byline string.

Additional information

Not applicable.

Related issues

Resolves #257 .

Removed .home__feed header style that was causing elements to flex and appear inline.
resolve #257
@jhung jhung added the bug Something isn't working label Feb 21, 2020
@jhung jhung requested a review from greatislander February 21, 2020 18:50
@jhung jhung self-assigned this Feb 21, 2020
@netlify
Copy link

netlify bot commented Feb 21, 2020

Deploy preview for pinecone ready!

Built with commit aad6845

https://deploy-preview-258--pinecone.netlify.com

Copy link
Collaborator

@greatislander greatislander left a comment

Choose a reason for hiding this comment

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

Works after fixing regression from dev.

@greatislander greatislander merged commit 738836b into dev Feb 21, 2020
@greatislander greatislander deleted the fix/byline-wrapping branch February 21, 2020 20:19
@greatislander greatislander added this to the 1.0.0-alpha.12 milestone Feb 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Byline can appear inline with the resource title
2 participants