search

cuny-academic-commons / cbox-theme

Default theme for Commons In A Box
GNU General Public License v2.0
20 stars 15 forks source link

Notifications Page displays oddly on CBOX 1.0.7 #186

Closed vothsco closed 10 years ago

vothsco commented 10 years ago

Help and Support members are noticing that the notification page goes off-center. Sam has provided a screen shot. Here is a link to the thread that has the screenshot.

http://commonsinabox.org/groups/help-support/forum/topic/notification-page-goes-off-center/

boonebgorges commented 10 years ago

@BoweFrankema Can you take a look at this? I don't understand exactly what I'm looking at. Everything is looking OK on my installations. @vothsco Do we have any idea what kind of customizations the users have done to their themes?

BoweFrankema commented 10 years ago

I'm on it :-)

On Fri, Feb 21, 2014 at 3:11 PM, Boone Gorges notifications@github.comwrote:

@BoweFrankema https://github.com/BoweFrankema Can you take a look at this? I don't understand exactly what I'm looking at. Everything is looking OK on my installations. @vothsco https://github.com/vothsco Do we have any idea what kind of customizations the users have done to their themes?

Reply to this email directly or view it on GitHubhttps://github.com/cuny-academic-commons/cbox-theme/issues/186#issuecomment-35732990 .

btalberg commented 10 years ago

Hi all, I ran into a corrupted notifications page as well. It looks like notification templates just got left out in Cbox 1.0.7. I copied the default notification templates in Buddypress into my Cbox child theme's 'members'->'single' folder hierarchy. I then overrode 'members'->'single'->'home.php' by copying Cbox's 'home.php' locally and adding the appropriate code:

48: elseif ( bp_is_user_notifications() ) : 49: locate_template( array( 'members/single/notifications.php' ), true ); 50:

boonebgorges commented 10 years ago

Ben - Please make sure that you've updated the theme to the latest version in addition to updating the plugin. Check Dashboard > Commons In A Box for a theme upgrade notice.

On 02/27/14 13:43, Ben Talberg wrote:

Hi all, I ran into a corrupted notifications page as well. It looks like notification templates just got left out in Cbox 1.0.7. I copied the default notification templates in Buddypress into my Cbox child theme's 'members'->'single' folder hierarchy. I then overrode 'members'->'single'->'home.php' by copying Cbox's 'home.php' locally and adding the appropriate code:

49: locate_template( array( 'members/single/notifications.php' ), true ); 50:

— Reply to this email directly or view it on GitHub https://github.com/cuny-academic-commons/cbox-theme/issues/186#issuecomment-36275850.

btalberg commented 10 years ago

Thanks Boone. For whatever reason the theme upgrade notice did not appear on my local install. I wonder if I accidentally ignored or deleted the notice during upgrade? Nonetheless, downloading the latest theme and dropping it into the theme folder resolved the notification page issue for me. Thanks again.

boonebgorges commented 10 years ago

Excellent. Whether it's a bug that's specific to your install or not, you definitely are not the first person to miss the fact that the theme has to be updated separately. This is a piece of workflow that we plan to improve in future versions.

Thanks for reporting the issue!

On 02/27/14 14:12, Ben Talberg wrote:

Thanks Boone. For whatever reason the theme upgrade notice did not appear on my local install. I wonder if I accidentally ignored or deleted the notice during upgrade? Nonetheless, downloading the latest theme and dropping it into the theme folder resolved the notification page issue for me. Thanks again.

— Reply to this email directly or view it on GitHub https://github.com/cuny-academic-commons/cbox-theme/issues/186#issuecomment-36279025.