By Altitudes on Friday, 03 April 2015
Posted in Technical Issues
Replies 26
Likes 0
Views 568
Votes 0
Hello

We have a styling issue on the module title in position es-groups-sidebar-top, as compared to es-dashboard-sidebar-top for example. See attachments.

Regards

Styling not fixed in Category files as of ES 1.4.8
Hi,

I have helped you fix this issue on your site. Please have a look.
DIR: /components/com_easysocial/themes/wireframe/groups/default.php

Hope this helps.
·
Friday, 03 April 2015 11:44
·
0 Likes
·
0 Votes
·
0 Comments
·
Hello

It's not working You can check by yourself going to any group on my site...
·
Friday, 03 April 2015 16:08
·
0 Likes
·
0 Votes
·
0 Comments
·
Your site is password protected and I do not have the login credentials for them.
·
Saturday, 04 April 2015 13:11
·
0 Likes
·
0 Votes
·
0 Comments
·
Hello Mark,

All access information is in "FORUM TOOLS" > "Update Site Details" on my side. Regarding .htaccess protection, this is in "Other details". I don't know how to make it more visible, because ES Support guys often ask me the same question...
·
Saturday, 04 April 2015 15:14
·
0 Likes
·
0 Votes
·
0 Comments
·
HI,

I am sorry for the late reply.
I have helped you fix this issue. Here is my screenshot: http://screencast.com/t/BESJA2iFv2Ns . Can you verify this issue is fixed for me?

Hope this helped and have a nice day.
·
Monday, 06 April 2015 10:37
·
0 Likes
·
0 Votes
·
0 Comments
·
This is fine, thank you
·
Monday, 06 April 2015 10:44
·
0 Likes
·
0 Votes
·
0 Comments
·
Hi,

I have apply the fix in this file: /components/com_easysocial/themes/wireframe/groups/item.php at line: 26 .

Have a nice day.
·
Monday, 06 April 2015 11:09
·
0 Likes
·
0 Votes
·
0 Comments
·
Thank you, but I think we have the same issue for a lot of module positions. I recently tested es-profile-editnotifications-sidebar-bottom position, and we have the same style problem.

Could you apply the correction everywhere it is needed?
·
Wednesday, 08 April 2015 13:36
·
0 Likes
·
0 Votes
·
0 Comments
·
Hi,

I am sorry as I can't assist you on this issue since our hands are pretty tied to our existing development right now but I will try assist you later once I have my free time. By the way, I have ask our designers to look into this issue and we will address this issue on the latest version of EasySocial.

I am sorry for the inconvenience caused.
·
Wednesday, 08 April 2015 17:48
·
0 Likes
·
0 Votes
·
0 Comments
·
Hello

OK. Could you at least commit the fix you did for position es-groups-sidebar-top? It was OK on my site after you corrected it, but this has been reverted after last ES update (1.3.25).

Thanks
·
Sunday, 12 April 2015 06:19
·
0 Likes
·
0 Votes
·
0 Comments
·
Hello Altitudes,

I am really sorry for the delay of this reply as it is a weekend for us here. Okay, I finally figured out why the "Other Details" isn't appearing for us and it's actually a bug in our template overrides on our forums. It's fixed now

I have applied a fix for you on your site and the fix is really simple, edited the file /components/com_easysocial/themes/wireframe/groups/item.php and at line 26, replaced the codes below,


<?php echo $this->render('module', 'es-groups-sidebar-top'); ?>


With,


<?php echo $this->render('module', 'es-groups-sidebar-top', 'site/dashboard/sidebar.module.wrapper'); ?>


Voila! It works now
·
Sunday, 12 April 2015 22:56
·
0 Likes
·
0 Votes
·
0 Comments
·
Thank you Mark, it's working fine. I loved seeing some French in your last sentence
·
Sunday, 12 April 2015 23:38
·
0 Likes
·
0 Votes
·
0 Comments
·
You are most welcome Haha, just got a little excited for a moment there but I have to get back to my coding cave right now
·
Sunday, 12 April 2015 23:40
·
0 Likes
·
0 Votes
·
0 Comments
·
Hello

I think there is another issue with this line of code...

Module position should be es-group-sidebar-top (with no 's' at group) and not es-groups-sidebar-top.... es-groups-sidebar-top (with an 's') is for the groups listing page.

Thanks
·
Monday, 27 April 2015 00:00
·
0 Likes
·
0 Votes
·
0 Comments
·
Hello Altitudes,

They should both have the 's' at the end for groups
·
Monday, 27 April 2015 01:09
·
0 Likes
·
0 Votes
·
0 Comments
·
But if you do so, any module you display on a group page will also be on the groups listing page (and vice versa ).

Is that really what is wanted?!
·
Monday, 27 April 2015 01:14
·
0 Likes
·
0 Votes
·
0 Comments
·
Yes, that is exactly the behavior What's wrong with that?
·
Monday, 27 April 2015 01:25
·
0 Likes
·
0 Votes
·
0 Comments
·
It's just that you may want to have different modules displayed between a group page and the groups listing page. I mean I do!

I need to display a module on group pages that has nothing to do on groups listing page.
·
Monday, 27 April 2015 01:40
·
0 Likes
·
0 Votes
·
0 Comments
·
Hello Altitudes,

Hurm, good point. To do this, you can go to this file: ../components/com_easysocial/themes/wireframe/groups/item.php and rename the es-groups-sidebar-top to es-group-sidebar-top. In the Module Manager, you can assign the module to position es-group-sidebar-top. Please give it a try.
·
Monday, 27 April 2015 11:03
·
0 Likes
·
0 Votes
·
0 Comments
·
Yes sure, that's what I did. Now the question is should you do this modification in EasySocial or should I do it by overriding item.php on my site?

It's up to you to decide! This would also apply to the position es-group(s)-sidebar-bottom. My opinion is this would be a good thing to do this for everybody Your customers probably did not use these positions before, otherwise they would have noticed the styling issue, so hopefully no one will be disturbed by this change.
·
Monday, 27 April 2015 14:08
·
0 Likes
·
0 Votes
·
0 Comments
·
And one more thing: we have the same kind of issue for events with module positions es-events-sidebar-top, es-events-sidebar-bottom (both styling issue and 's' issue to have different modules between event details page and events listing page).

I'll need to use these positions very soon and I would be glad if they are fixed too!

So to summarize...

Styling should be fixed in:
/components/com_easysocial/themes/wireframe/groups/item.php (done)
/components/com_easysocial/themes/wireframe/groups/default.php
/components/com_easysocial/themes/wireframe/groups/category.php
for positions es-group(s)-sidebar-top, es-group(s)-sidebar-bottom

Styling should be fixed in:
/components/com_easysocial/themes/wireframe/events/item.php
/components/com_easysocial/themes/wireframe/events/default.php
/components/com_easysocial/themes/wireframe/events/category.php
for positions es-event(s)-sidebar-top, es-event(s)-sidebar-bottom

es-events-sidebar-top, es-events-sidebar-bottom should be changed to es-event-sidebar-top, es-event-sidebar-bottom (no 's') in /components/com_easysocial/themes/wireframe/events/item.php if you agree.

es-groups-sidebar-top, es-groups-sidebar-bottom should be changed to es-group-sidebar-top, es-group-sidebar-bottom (no 's') in /components/com_easysocial/themes/wireframe/groups/item.php if you agree.
·
Monday, 27 April 2015 14:47
·
0 Likes
·
0 Votes
·
0 Comments
·
Hello Altitudes,

Thanks for the suggestion. I can't decide it on my own. I need to discuss with developers team. In a mean time, you can write it in our Voices page here: http://www.stackideas.com/voices/easysocial.
·
Monday, 27 April 2015 17:31
·
0 Likes
·
0 Votes
·
0 Comments
·
Let me know what the developers team says, and then if this is required I will write something in Voices
·
Monday, 27 April 2015 17:40
·
0 Likes
·
0 Votes
·
0 Comments
·
We'll see what we can do for this on the next updates.
·
Wednesday, 29 April 2015 00:37
·
0 Likes
·
0 Votes
·
0 Comments
·
Hello

I updated to ES 1.3.30 recently and see that position names were corrected (e.g. es-groups-sidebar-top was changed to es-groups-item-sidebar-top)

But styling issue was only partially fixed (in /components/com_easysocial/themes/wireframe/groups/item.php and default.php but not in other files I listed in my previous post). Any hope to have all the files fixed?

Thank you
·
Saturday, 08 August 2015 05:38
·
0 Likes
·
0 Votes
·
0 Comments
·
Looks like we missed this. This will be fixed on the next release
·
Saturday, 08 August 2015 14:24
·
0 Likes
·
0 Votes
·
0 Comments
·
View Full Post