Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Weblinks conflict with Aside section + other error #224

Closed
csbalazs opened this issue May 11, 2015 · 3 comments
Closed

Weblinks conflict with Aside section + other error #224

csbalazs opened this issue May 11, 2015 · 3 comments

Comments

@csbalazs
Copy link

Hi,

  • If I use the core Weblinks component (with List All Web Link Categories type) and I set 3 columns preset, then the Aside is appears under the Sidebar section.

Joomla! 3.4.1
Weblinks 3.4.0
Gantry 5.0.0 RC1
Hydrogen 5.0.0 RC1

2015 0511-01
2015 0511-02
2015 0511-03
2015 0511-04

  • If I use the core Weblinks component (with Submit a Web Link type) than I get this one: Maximum function nesting level of '100' reached, aborting!

2015 0511-06

@csbalazs csbalazs changed the title Weblinks conflict with Section Aside Weblinks conflict with Aside section + other error May 11, 2015
@w00fz
Copy link
Member

w00fz commented May 11, 2015

What browser did you use there? It shouldn't be collapsing like that.

@w00fz w00fz added the question label May 11, 2015
@w00fz w00fz added this to the Release Candidate 2 milestone May 11, 2015
@csbalazs
Copy link
Author

All browsers have the same problems.

@w00fz w00fz added invalid and removed question labels May 12, 2015
@w00fz
Copy link
Member

w00fz commented May 12, 2015

This is not a Gantry issue, Weblinks appears to be rendering an unnecessary closing </div> that breaks the layout of the template. If you edit the file components/com_weblinks/views/categories/tmpl/default.php you'll see at the end of the file the extra </div>. If you remove that the columns don't collapse anymore.

It also appears they already fixed it (see joomla-extensions/weblinks@d3b6ae#diff-cdca3a), so presumably it will be available in their next release.

Refs:

Thanks.

@w00fz w00fz closed this as completed May 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants