Open-event-frontend: Replace anchor tags with link-to helpers for transition within the app.

[anu-007]

I'm submitting a ... (check one with "x")

• [ ] bug report
• [x] feature request
• [ ] support request => Please do not submit support requests here, instead ask your query in out Gitter channel at https://gitter.im/fossasia/open-event-frontend

currently there are many places hard coded anchor tag links are there for transition within the project they should be replaced by link-to helper as anchor tags reloads the app instead of just transitioning.

[anu-007]

I like to work on this.

[niranjan94]

Can you tell me which cases?

[niranjan94]

Because as far as I can see achor tags react the same way as link-to (Which are just basically anchor tags along with some _magic_ to add the active depending on which page the user is in) as long as the links are relative to the project and lead to known routes.

There is a reason why some places don't use link-to helper and use a normal a tag along with a href-to instead. The link-to tag creates a whole new component and hence all the bindings, memory usage etc that comes along with a component is also there for every link. But href-to on the other hand is a simple helper and just provides a link text. Both href-to and link-to are similar is all aspects .. except for the fact that link-to provides automatic active class binding.

But we don't need the active class binding at all places. So, as far as this project is concerned, anywhere you need automatic active class binding, use link-to else use a normal a tag along with the href-to helper.

[niranjan94]

@fossasia/open-event-frontend please note. 👆

[niranjan94]

currently there are many places hard coded anchor tag links are there for transition within the project they should be replaced by link-to helper as anchor tags reloads the app instead of just transitioning.

This shouldn't be happening anywhere. But some routes related to authentication, like logout for example will cause a full page reload.

[anu-007]

@niranjan94 thanks for the clarification and I completely agreed with you, most of the places that I find out would not need active class binding hence closing this issue.