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

Spacing for post type and comments #12

Open
amongthestones opened this issue Nov 14, 2018 · 6 comments
Open

Spacing for post type and comments #12

amongthestones opened this issue Nov 14, 2018 · 6 comments
Labels
enhancement New feature or request

Comments

@amongthestones
Copy link

I recently found Delivering WordPress in 7KB and really like the theme for its low bandwidth and its minimal look. But there are spaces on either side of post types and also the Leave a Comment link. Were these intentional? I was looking to delete them but I haven’t found the line (not much of a coder in spite of the GitHub account).

@jacklenox
Copy link
Owner

@amongthestones Thank you for reporting this. Can you give a bit more detail? I'm not sure I follow exactly what you mean. Could you maybe add a screenshot?

@jacklenox jacklenox added the question Further information is requested label Nov 23, 2018
@amongthestones
Copy link
Author

Here's a screenshot of what I mean. There are two spaces on either side of the post type (in this case, Links), and the leave a comment section.
screenshot 2019-01-14 at 16 59 58

@sgclark
Copy link

sgclark commented Mar 2, 2019

I too recently adopted the Susty theme and have similar questions about the "Meta" details for posts. I can't seem to find the PHP file that contains these details. I too would like to make some edits and changes to these items. I also wanted to put the "Leave A Comment" link on a different line than the "Tags" and maybe move those "Posted" and "Tagged" meta data details up to the "Posted" and "Author" details at the top of the post. Anyway, really just curious where the file is that I can make these changes to.

My site is www.sgclark.com if you want to check it out.

Thanks!

@jacklenox
Copy link
Owner

Hi all, so sorry for not replying on this sooner. @amongthestones So yeah, this is a super weird design decision that I made. I don't really know why. 🤷‍♂️ I kind of wanted them to be slightly button-like, but on reflection it's a bit rubbish! The padding is added here:

susty/style.css

Lines 930 to 933 in 7a43aa1

.cat-links a, .comments-link a, .edit-link a {
padding: 0 .5em;
margin: 0 .5em 0 0;
}

@sgclark Nice child theme! It looks as though you solved the issue yourself, but let me know if you have any other questions.

@jacklenox jacklenox added enhancement New feature or request and removed question Further information is requested labels May 28, 2019
@sgclark
Copy link

sgclark commented May 28, 2019

Yes, I did eventually figure it out but what I did does get a little ‘moody’ on different sized devices. I’ll compare your code above to what I did to see if it behaves better (which I am sure it will).

Thanks, re: my site. I love how damn quick the base Susty code is when the site loads and even with the additional things I have on my site (all the images/galleries and the mobile nav treatment), it still flies!!

@sgclark
Copy link

sgclark commented May 31, 2019

Finally got around to looking at the "Meta Tags" code I'm using compared to what you noted above.

The weird thing I am seeing was that the font size of the links in the "meta tags" rendered at a smaller size than the 'plain' text in the 'meta tags'. I used the font-size: inherit css to normalize the links in my CSS to match the size of the plain text but I was wondering if there is some font size control in the master Susty CSS that was controlling that? I'm also seeing some extra spacing/padding before and after the .cat-links links.

Not a big deal but curious none the less.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants