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

Are anchored link supported? #13

Closed
llahnoraa opened this Issue Aug 26, 2016 · 6 comments

Comments

Projects
None yet
2 participants
@llahnoraa

When someone click a link with anchor and take them to another page, is it possible to expand that section open?

@nico3333fr nico3333fr self-assigned this Aug 26, 2016

@nico3333fr

This comment has been minimized.

Show comment
Hide comment
@nico3333fr

nico3333fr Aug 26, 2016

Owner

Hi @llahnoraa

damn, I've thought to this case with my tabpanel script, but not with this plugin :)
For the moment, there is only an option to make it open by default, see http://a11y.nicolas-hoffmann.net/hide-show/

I'll have a look asap.

Owner

nico3333fr commented Aug 26, 2016

Hi @llahnoraa

damn, I've thought to this case with my tabpanel script, but not with this plugin :)
For the moment, there is only an option to make it open by default, see http://a11y.nicolas-hoffmann.net/hide-show/

I'll have a look asap.

@llahnoraa

This comment has been minimized.

Show comment
Hide comment

+1

@nico3333fr

This comment has been minimized.

Show comment
Hide comment
@nico3333fr

nico3333fr Feb 21, 2017

Owner

Hi @llahnoraa ,

could you test ? It is okay for me https://a11y.nicolas-hoffmann.net/hide-show/#anims (test with or without the fragment) but I prefer another test to confirm. ;)

Cheers,
Nicolas

Owner

nico3333fr commented Feb 21, 2017

Hi @llahnoraa ,

could you test ? It is okay for me https://a11y.nicolas-hoffmann.net/hide-show/#anims (test with or without the fragment) but I prefer another test to confirm. ;)

Cheers,
Nicolas

@llahnoraa

This comment has been minimized.

Show comment
Hide comment
@llahnoraa

llahnoraa Feb 22, 2017

It worked. Looks good to me so far. Can anyone else test on this, such as mobile-wise?

Do we need to update the plugin file for this?

And thank you! (:

llahnoraa commented Feb 22, 2017

It worked. Looks good to me so far. Can anyone else test on this, such as mobile-wise?

Do we need to update the plugin file for this?

And thank you! (:

@nico3333fr

This comment has been minimized.

Show comment
Hide comment
@nico3333fr

nico3333fr Feb 22, 2017

Owner

Yes, as you may see in the diff, 942cd06 there are not many differences.

Owner

nico3333fr commented Feb 22, 2017

Yes, as you may see in the diff, 942cd06 there are not many differences.

@llahnoraa

This comment has been minimized.

Show comment
Hide comment
@llahnoraa

llahnoraa Mar 31, 2017

Perfect! Thank you for your help on this!

Perfect! Thank you for your help on this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment