kevinsguides / auto-scrollspy

Easy Local Navigation for Joomla
https://kevinsguides.com/guides/webdev/joomla/free-extensions/autoscrollspy
4 stars 1 forks source link

scrollspy and Gantry5 #1

Open johanpeters opened 7 months ago

johanpeters commented 7 months ago

Hello, i just found your plugin.. just what i was looking for. i would like to use it on a gantry5 framework based website.

but it does not show.. i configured it, assigned to a module position, menu assignment but its not working. it does not show the module at all.

should it work? i have a blog page menu item.. and would like to have this module shown when you open one of the published articles. the article has headings assigns.. plugin is enabled..

kevinsguides commented 7 months ago

Thanks for the comment. I clearly didn't test this enough! The module doesn't load for me either when I use it on many third-party templates. I only tested it on the default template... oops. Please expect an update/fix shortly.

kevinsguides commented 7 months ago

The issue is "hopefully" resolved with newest update, 1.2.1, please let me know if you still encounter issues.

johanpeters commented 7 months ago

Hello Kevin

it shows the module now.. but not working completely.

  1. floating not working
  2. sticky not working

article headers not visable / linkable. the default settings does no show headers at al. when i change to list for example is shows numbers

[image: image.png]

On Thu, 1 Feb 2024 at 02:18, Kevin Olson @.***> wrote:

The issue is "hopefully" resolved with newest update, 1.2.1, please let me know if you still encounter issues.

— Reply to this email directly, view it on GitHub https://github.com/kevinsguides/auto-scrollspy/issues/1#issuecomment-1920298423, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKYSGSIAHHQW2QGH4RS6JTYRLUPFAVCNFSM6AAAAABCT3C4I6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRQGI4TQNBSGM . You are receiving this because you authored the thread.Message ID: @.***>

kevinsguides commented 7 months ago

I will release another update later today which hopefully resolves all module output issues, and fixes another minor problem with how I updated it yesterday. For me, the floating panel does work with Gantry5 and their free helium template... I am trying to make sticky module work better, but really this is something that's better handled by whatever template you're using.

Please wait for the update later today and see if that helps. If you'd like in the mean time...

johanpeters commented 7 months ago

Hello Kevin,

  1. article has several headings
  2. article has more headings then set in the plugin as minimum

see screenshot.. the index in the article is a plugin {index} that reads the heading and creates the index. so you see, i have 17 article headings.

on the right contentspy also displays 17 numbers but without text. when i use default settings no entries ar shown.. when i select number layout this is the result.

kevinsguides commented 7 months ago

If you're referring to the RegularLabs quick index plugin, I cannot make that compatible with this. You'll want to disable one or the other, since AutoSS also indexes all the headings in your article, in order to make it work. They both manipulate the headings and the article output. Quick index plugin also seems to break the floating panel option. I've enhanced how the sticky module position feature works, but it does not seem to play nicely with the Gantry template I'm testing on. Sticky module positions are really something better left to the template developer, as I cannot create work arounds for every template out there.

Also, not sure that screenshot is what you meant to share? Looks like webmail history.

Latest update I just released has a few other improvements but I don't think they'll resolve your issue. If you're not using the quick index plugin, please let me know what other plugins you're using that affect headings and articles.

Hope that makes sense, and thanks for the feedback. If you happen to run into more problems with the other index plugin disabled, let me know.

johanpeters commented 7 months ago

Hello Kevin

it works now as soon as i remove the {index} from regular labs. also floating works nicely.

only 1 thing .. the heading is on top of the page .. would be nice to have a setting to respect the template see screenshot.. to the selected heading should be 100px from top for example.

with regards

On Fri, 2 Feb 2024 at 02:04, Kevin Olson @.***> wrote:

If you're referring to the RegularLabs quick index plugin, I cannot make that compatible with this. You'll want to disable one or the other, since AutoSS also indexes all the headings in your article, in order to make it work. They both manipulate the headings and the article output. Quick index plugin also seems to break the floating panel option. I've enhanced how the sticky module position feature works, but it does not seem to play nicely with the Gantry template I'm testing on. Sticky module positions are really something better left to the template developer, as I cannot create work arounds for every template out there.

Also, not sure that screenshot is what you meant to share? Looks like webmail history.

Latest update I just released has a few other improvements but I don't think they'll resolve your issue. If you're not using the quick index plugin, please let me know what other plugins you're using that affect headings and articles.

Hope that makes sense, and thanks for the feedback. If you happen to run into more problems with the other index plugin disabled, let me know.

— Reply to this email directly, view it on GitHub https://github.com/kevinsguides/auto-scrollspy/issues/1#issuecomment-1922599152, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKYSGS6QJXJP43TIQZB5ZLYRQ3QTAVCNFSM6AAAAABCT3C4I6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4TSMJVGI . You are receiving this because you authored the thread.Message ID: @.***>

johanpeters commented 7 months ago

Schermafbeelding 2024-02-02 om 08 23 45

johanpeters commented 7 months ago

regarding the position.. if it respacts the fix header position.. than it should be okay.

currently the header selected is on top of the page.. would be better to have a option to configure this..

On Fri, 2 Feb 2024 at 02:04, Kevin Olson @.***> wrote:

If you're referring to the RegularLabs quick index plugin, I cannot make that compatible with this. You'll want to disable one or the other, since AutoSS also indexes all the headings in your article, in order to make it work. They both manipulate the headings and the article output. Quick index plugin also seems to break the floating panel option. I've enhanced how the sticky module position feature works, but it does not seem to play nicely with the Gantry template I'm testing on. Sticky module positions are really something better left to the template developer, as I cannot create work arounds for every template out there.

Also, not sure that screenshot is what you meant to share? Looks like webmail history.

Latest update I just released has a few other improvements but I don't think they'll resolve your issue. If you're not using the quick index plugin, please let me know what other plugins you're using that affect headings and articles.

Hope that makes sense, and thanks for the feedback. If you happen to run into more problems with the other index plugin disabled, let me know.

— Reply to this email directly, view it on GitHub https://github.com/kevinsguides/auto-scrollspy/issues/1#issuecomment-1922599152, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKYSGS6QJXJP43TIQZB5ZLYRQ3QTAVCNFSM6AAAAABCT3C4I6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4TSMJVGI . You are receiving this because you authored the thread.Message ID: @.***>

kevinsguides commented 7 months ago

I will review and get back to you

johanpeters commented 7 months ago

I will review and get back to you

greate Kevin.. currently the plugin place the header on top of the page.. its not readable on my site.. but for most sites with fixed headers) it will not work. so, if possible it would be nice that the position of the header selected / active is not on top of the screen but for example 100px (to configure) padding on top..

so that the current header section is not completly on top, but a little bit below that.

if needed i can explain with a example image..

kevinsguides commented 7 months ago

Ah, I think I understand what you mean. Yes, that's an important feature I meant to add. I'll try to have something in the next day or two. Thanks