ThemeAwesome / WP-Forge

WP-Forge - A Foundation for Sites WordPress Theme
https://themeawesome.com/wp-forge-wordpress-foundation-for-sites-theme/
85 stars 28 forks source link

Customizer Header Settings #53

Closed badcat closed 7 years ago

badcat commented 7 years ago

Hi - In Customizer > Header Settings > Site Identity under Tagline section.

Seems like "Display Site Title and Tagline" is duplicative to the above options; "Hide site title" and "Hide site tagline"? I'd think we'd only need the first two and not the third, no?

customizer

There's also some capitalization issue :)

thanks.

tsquez commented 7 years ago

One is for the custom logo the other is for the custom header... when you have both they both appear... trying to figure out how to remove one... sand the capitalization is a matter of preference yah grammer nazi... lol

via my phone

On Aug 22, 2017 8:17 PM, "BadCat Design" notifications@github.com wrote:

Hi - In Customizer > Header Settings > Site Identity under Tagline section.

Seems like "Display Site Title and Tagline" is duplicative to the above options; "Hide site title" and "Hide site tagline"? I'd think we'd only need the first two and not the third, no?

[image: customizer] https://user-images.githubusercontent.com/26096/29597308-d2782788-878f-11e7-85e8-4456b9780576.png

There's also some capitalization issue :)

thanks.

β€” You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ThemeAwesome/wp-forge/issues/53, or mute the thread https://github.com/notifications/unsubscribe-auth/AB4nNbtOQOrbt6Nt_Cq62fpv9hO4WwR4ks5sa5ncgaJpZM4O_cGD .

tsquez commented 7 years ago

Forgot to mention, the new theme only uses the custom logo, not both...a lot of surprises coming amigo... toy gonna love it ;)

via my phone

On Aug 22, 2017 8:17 PM, "BadCat Design" notifications@github.com wrote:

Hi - In Customizer > Header Settings > Site Identity under Tagline section.

Seems like "Display Site Title and Tagline" is duplicative to the above options; "Hide site title" and "Hide site tagline"? I'd think we'd only need the first two and not the third, no?

[image: customizer] https://user-images.githubusercontent.com/26096/29597308-d2782788-878f-11e7-85e8-4456b9780576.png

There's also some capitalization issue :)

thanks.

β€” You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ThemeAwesome/wp-forge/issues/53, or mute the thread https://github.com/notifications/unsubscribe-auth/AB4nNbtOQOrbt6Nt_Cq62fpv9hO4WwR4ks5sa5ncgaJpZM4O_cGD .

badcat commented 7 years ago

Hmm - seems to exhibit strange hiding behavior without using Custom Logo or Custom Header (img). Maybe it just needs to be labeled more clearly? Not sure exactly what that would be though. Er.. thoughts?

LOL - yeah I was mentioning the capitalization only because the items were handled differently and they were directly under the other choices. (rib rib... and you spelled grammar all funny too :)

badcat commented 7 years ago

Also... you mention, "Forgot to mention, the new theme only uses the custom logo, not both." But the Customizer > Header Settings > Header Image is still there and allows both -- is the Custom Header Image really going away?

tsquez commented 7 years ago

Yes, it's basically the same thing, just one is called custom header and other custom logo. The custom logo is easier to implement. Read up on it.

The only reason both are there is because, when I took away the custom header and added the custom logo a few users complained because it destroyed the headers they had built.

So I added the custom header back into the mix and that is why you see so I added the custom header back into the mix and that is why you see duplicate site title and site description.

I have been trying to figure out a way how to use just one instance of the site title and site description but I got busy working on the new theme and completely forgot about that until you brought it up... LOL.

Have you heard of generatepress? If you haven't look into that theme and then think about generatepress using Foundation.

via my phone

On Aug 22, 2017 9:59 PM, "BadCat Design" notifications@github.com wrote:

Also... you mention, "Forgot to mention, the new theme only uses the custom logo, not both." But the Customizer > Header Settings > Header Image is still there and allows both -- is the Custom Header Image really going away?

β€” You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ThemeAwesome/wp-forge/issues/53#issuecomment-324221263, or mute the thread https://github.com/notifications/unsubscribe-auth/AB4nNUAo4C2cFv4F030YgWUmJHiow1Gkks5sa7GUgaJpZM4O_cGD .

badcat commented 7 years ago

Yes - I'm pretty familiar with GP. I've built about a dozen sites with it this year :) You'll see me lurking around Tom's GitHub and Forums.

Also yes to Kirki of course; Was playing with this.. https://www.youtube.com/watch?v=Bqv175VbOM0 I assume many of the page builders out there will continue the move to Kirki or similar. And gotta love this - http://kirki-build.aristath.com

Looking forward to seeing what you're working on πŸ‘

tsquez commented 7 years ago

Cool cool, like I said, just think of GeneratePress running on Foundation for Sites with Kirki built in….so far I think it’s pretty damn cool πŸ˜ƒ

badcat commented 7 years ago

I'm game - show it :) Got a name picked yet?

tsquez commented 7 years ago

Hahahahah yeah I can shot it to yah and yes I have a name for it. If you want a look hit me up on my personal email and I'll send you the link to the demo πŸ˜‰ ...gonna close this πŸ‘