Automattic / Automattic/default-small-business-theme

Option to hide homepage title - issues roundup.

Open
#8 1 comment 0 reactions 1 assignee Claimed by @crunnells View on GitHub
Dominant language
SCSS
Stars
10
Forks
8
PR merge metrics
No merged PRs in 30d

Description

I did some testing of this feature before deploying, and found some minor issues:

* I'm seeing a space above the cover image, when it's the first element on the page
![image](https://user-images.githubusercontent.com/4550351/48923519-58e2d080-ef03-11e8-9625-686cf7f73c4f.png)
In general the space should not be there, if the first element of the content is an image, gallery, cover, or video.

* If a page created with classic editor is used, there is not space above the first element
![image](https://user-images.githubusercontent.com/4550351/48923527-68621980-ef03-11e8-8d5d-379a450c4421.png)

* I was wondering if we should change the option name from "Hide Front Page Title" to "Hide homepage Title", since in the select above it, it's called "Homepage"

cc @crunnells

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.