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

router-link exact path matching (#fix 2040) #2278

Closed
wants to merge 4 commits into from
Closed

router-link exact path matching (#fix 2040) #2278

wants to merge 4 commits into from

Conversation

samgre
Copy link

@samgre samgre commented Jul 5, 2018

There have been at least two requests (#2040 and #2193) for adding ability to match only the path in a router-link.

A use case is highlighted in the issue #2040. Sometimes you want to highlight a router-link regardless of the query string parameters. This happens e.g. with paginated listing pages, where the desired page number is often supplied as a query string parameter.

This pull requests implements the requested feature.

@samgre
Copy link
Author

samgre commented Jul 5, 2018

Thanks @curtisbelt for notifying me. Made the pull request!

@posva
Copy link
Member

posva commented Aug 10, 2018

Note to myself: having a long list of classes is quite confusing, maybe we should refactor this in some kind of string based prop and an object of classes in config like linkClasses

@JulienTant
Copy link

Any news on this PR ?

@1337huania
Copy link

Waiting for this PR too... Is there any news?

@MariaSMM
Copy link

MariaSMM commented Feb 1, 2019

Hey guys, have you got any solutions?

@Djaler
Copy link

Djaler commented Mar 14, 2019

What about it?

@vuejs vuejs deleted a comment from gaofeng0630 Jun 5, 2019
@posva posva added this to To Do in v3 pre v4 May 19, 2020
@samgre samgre closed this Jun 20, 2020
@posva posva moved this from To Do to Done in v3 pre v4 Dec 29, 2020
@posva posva added this to Done in 3.x Jun 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
3.x
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

6 participants