Project

General

Profile

Feature #2924

Make header of Help blog a clickable link to the homepage

Added by Matt Gold over 5 years ago. Updated over 4 years ago.

Status:
Resolved
Priority name:
Normal
Category name:
WordPress (misc)
Target version:
Start date:
2013-12-08
Due date:
2014-03-11
% Done:

0%

Estimated time:

Description

Dom, can you please help Scott make the Commons logo on the Help site - http://help.commons.gc.cuny.edu/ - a clickable link to the homepage of the Commons? Many thanks.


Related issues

Related to CUNY Academic Commons - Feature #2080: Upgrade Woo Canvas to Current Version (5.0) - ResponsiveResolved2012-08-282014-03-11

History

#1 Updated by scott voth over 5 years ago

  • Assignee changed from Daniel Bachhuber to Dominic Giglio

Hi Dom - I think this will probably entail create a child theme of Canvas that has a header with two links - one to to the Home page and the second to Help & Support, unless you have a better solution.

#2 Updated by Matt Gold over 5 years ago

Oops! thanks for correcting the assignee, Scott.

#3 Updated by Dominic Giglio over 5 years ago

  • Target version set to 1.5.11

Scott,

It appears that you are correct. Because this is a WooTheme (Canvas) we will probably need to customize the functionality through a child. Premium themes, like those from Woo, are awesome for getting up and running quickly and then having really nice customization features in the admin once the site is running. But, like any framework, you are then required to work within the constraints created by the framework (I run into this when developing on Rails all the time).

The current logo in that header is not just an image, it is set as the CSS background for the header area within the Canvas theme's admin settings. This is why we can't just "make it a link." We will need to override the header area in a child theme and manually create our own version so that we can keep a link to the Help site's homepage (which is the link on the right-hand side right now) as well as a link to the commons home page using our logo on the left.

I think we should take this opportunity to close out issue #2080 (Upgrade Woo Canvas to Current Version) since we've received the budget money that that issue has been waiting for. I believe customizing the old version currently installed through a child theme will make closing that issue much harder. We should upgrade your theme, then see if there are any new Woo features that will allow us to create a link in the header - and if not we can then whip up a quick child theme and customize it manually if need be.

#4 Updated by Dominic Giglio over 5 years ago

  • Target version changed from 1.5.11 to 1.5.12

#5 Updated by Dominic Giglio over 5 years ago

  • Target version changed from 1.5.12 to 1.5.13

#6 Updated by Boone Gorges over 5 years ago

  • Target version changed from 1.5.13 to 1.5.14

#7 Updated by Boone Gorges over 5 years ago

  • Target version changed from 1.5.14 to 1.5.15

#8 Updated by Dominic Giglio about 5 years ago

  • Target version changed from 1.5.15 to 1.5.16

#9 Updated by Boone Gorges about 5 years ago

  • Target version changed from 1.5.16 to 1.5.17

#10 Updated by Boone Gorges about 5 years ago

  • Target version changed from 1.5.17 to 1.5.18

#11 Updated by Boone Gorges about 5 years ago

  • Target version changed from 1.5.18 to 1.5.19

#12 Updated by Boone Gorges about 5 years ago

  • Target version changed from 1.5.19 to 1.5.20

#13 Updated by Matt Gold about 5 years ago

Let's definitely try to make this part of 1.5.20. Thanks, Dom!

#14 Updated by Matt Gold about 5 years ago

  • Due date set to 2014-03-11

#15 Updated by Boone Gorges about 5 years ago

  • Target version changed from 1.5.20 to 1.5.21

#16 Updated by Boone Gorges about 5 years ago

  • Target version changed from 1.5.21 to 1.5.22

#17 Updated by Boone Gorges about 5 years ago

  • Target version changed from 1.5.22 to 169

#18 Updated by Boone Gorges about 5 years ago

  • Target version changed from 169 to 1.6.1

#19 Updated by Boone Gorges almost 5 years ago

  • Target version changed from 1.6.1 to 1.6.2

#20 Updated by Boone Gorges almost 5 years ago

  • Target version changed from 1.6.2 to 1.6.3

#21 Updated by Boone Gorges almost 5 years ago

  • Tracker changed from Bug to Feature

#22 Updated by Boone Gorges almost 5 years ago

  • Target version changed from 1.6.3 to 1.6.4

#23 Updated by Boone Gorges almost 5 years ago

  • Target version changed from 1.6.4 to 1.6.5

#24 Updated by Boone Gorges almost 5 years ago

  • Target version changed from 1.6.5 to 1.6.6

#25 Updated by Dominic Giglio almost 5 years ago

Scott,

Not sure if you're still on vacation, I'll just wait to hear from you whenever you're back or can respond. I've finally gotten around to addressing this issue for you. I've created a child theme that will be available to you in the Admin after the next release. I've overridden the woo_logo() function, which is responsible for rendering the contents of the Canvas header, to also display an image back to the Commons homepage as an image.

I'm attaching a screenshot so you can see what it looks like on my local machine to let me know what you think. I'm not sure why the rest of the page layout/styling is off/broken, but that shouldn't be a problem seeing as you're running the latest Canvas version on the live site and it's working without issue. There are probably associated files that don't exist in my environment - could even be as simple as my local environment does not have the same set of plugins activated.

Let me know if this is satisfactory or if you want further changes made before the next release.

#26 Updated by Matt Gold almost 5 years ago

Thanks very much for your work on this, Dom.

#27 Updated by Boone Gorges almost 5 years ago

  • Target version changed from 1.6.6 to 1.6.7

#28 Updated by Boone Gorges almost 5 years ago

  • Target version changed from 1.6.7 to 1.6.8

#29 Updated by scott voth almost 5 years ago

Hi Dom -
Sorry I missed this (was in la-la, jet-lag land, I think). Thanks for following up. Can I try to configure the child theme on a test site to confirm everything is OK? I tried to install it - but realize I don't know what it's called. Can you provide the child theme's name? Or maybe I don't have access to it yet? Thanks!

#30 Updated by Dominic Giglio almost 5 years ago

No need to apologize Scott, I'm the one causing this issue to take forever! :-)

We should definitely take a look at the theme on a test site first. Do you have an existing test site in mind? Are you going to create a new one?

The name of the theme is: "Help and Support Canvas." Let me know which test site you would like to try it out on and I will enable it on that site. Once we know it's working I will enable it on the official Help and Support site.

Hope you had a great vacation, looking forward to finally trying to complete this for you.

#31 Updated by scott voth almost 5 years ago

Thanks Dom! I have created a new test site called http://testhelp.commons.gc.cuny.edu/
Can you activate the child theme on this one? Thanks!

#32 Updated by Boone Gorges almost 5 years ago

  • Target version changed from 1.6.8 to 1.6.9

#33 Updated by Dominic Giglio almost 5 years ago

Scott,

I've enable (not activated) the new "Help and Support Canvas" child theme on the new test site. I didn't want to activate it for you. Just go into the dashboard and you can find the new child in the themes admin page.

Let me know how the testing goes. Canvas is a pretty advanced theme, I could very well have missed something or configured the child incorrectly. Now that we've got the child in there, we should be able to wrap this up in no time.

#34 Updated by scott voth almost 5 years ago

Hi Dom -
I was able to activate the child theme, and I imported content over from the prod site. It looks like there is an issue with the header - probably something simple like a missing div or something - but the site name is getting pushed down (should be on same level with the CAC logo), and the primary navigation is also getting pushed below the header. I disabled the slider, which was also feeling the affects, so we can better analyze the issue. If you can take a look that would be great. Here is the link - http://testhelp.commons.gc.cuny.edu

Thanks.

#35 Updated by Boone Gorges almost 5 years ago

  • Target version changed from 1.6.9 to 1.6.10

#36 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.6.10 to 1.6.11

#37 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.6.11 to 1.6.12

#38 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.6.12 to 1.6.13

#39 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.6.13 to 1.6.14

#40 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.6.14 to 1.6.15

#41 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.6.15 to 1.6.16

#42 Updated by scott voth over 4 years ago

Hi Dom - I did some more work on this today and got the styling pretty good now, but am still having the issue of the menu getting pushed down into the content area. The slider too seems to be reacting to the issue. (I turned it off for now.)

Now that you are back from vacation - can you take a look - or maybe we can take a look together at this Friday's meeting? Thanks a lot.

Here's the link: http://testhelp.commons.gc.cuny.edu

#43 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.6.16 to 1.6.17

#44 Updated by Matt Gold over 4 years ago

Hi Dom -- did you and Scott make any headway on this last Friday in our meeting?

#45 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.6.17 to 1.7.1

#46 Updated by Dominic Giglio over 4 years ago

Scott,

I think I've got this figured out. I won't bore you with the technical details, but basically my child theme wasn't pulling all the required styes from the Canvas parent. That was what was throwing off our layout while testing.

I've pushed up some new code, our next release is on Saturday. So lets have a look at your testhelp.commons.gc.cuny.edu site Saturday or Sunday to see if this new code takes care of it. We may need one more round to add custom CSS if we can't add the appropriate styles in the Dashboard - but I think this code gets us REALLY close.

#47 Updated by Boone Gorges over 4 years ago

Dom - Now that the latest updates have been on the Commons for a while, can we try to get this ticket closed?

#48 Updated by Dominic Giglio over 4 years ago

Definitely. Scott is taking a look at the styling and layout now so we can decide if the new structure is appropriate.

I placed the two new header links inside of a column class provided by Canvas and I asked Scott if that was what he wanted or if he'd rather I place them next to each other so we can style them more easily with custom CSS in the Dashboard.

As soon as I hear from him I can make any changes he needs or close out this ticket if he's good to go.

#49 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.7.1 to 1.7.2

#50 Updated by scott voth over 4 years ago

Hi Dom - Sorry I was preoccupied with other stuff, but I just got a chance to format the test site using the new child theme and everything looks great! On my end, I need get the slider image heights consistent, but I can work on that (Canvas seems to have changed some stuff.)

So how do we go forward? There are actually two sites that need to use the child theme (http://codex.commons.gc.cuny.edu/ and http://help.commons.gc.cuny.edu/). And once we shift over, it seems like there might be some customization on my end.

Can you make the child theme available to me, so I can switch it on and off as I configure the two sites?

Thanks!

#51 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.7.2 to 1.7.3

#52 Updated by Dominic Giglio over 4 years ago

Scott, I've enabled the "Help and Support Canvas" child theme on both the Help and Codex sites so you can switch back and forth on your own now. If you create any test sites that need the theme enabled just let me know here and I'll turn it on.

When we release next week, on the 11th, lets revisit this ticket and close it out if you have no further issues. We can always open more tickets in the future if we need to continue working on this.

#53 Updated by Boone Gorges over 4 years ago

  • Target version changed from 1.7.3 to 1.7.4

#54 Updated by Dominic Giglio over 4 years ago

  • Status changed from Assigned to Resolved

This one's for you Boone. RESOLVED!

Also available in: Atom PDF