Abdón Rodríguez

Remove the HTML Imports stuff

1

I think we can simplify the project removing the HTML Imports stuff

Convert project to TypeScript

1

With this conversion, we can also drop the `@polymer/gen-typescript-declarations` dependency. ### Migration - [ ] Convert tests - #362 - [ ] Convert API docs and demos #363 - [...

I want to propose migrate from [polyserve](https://github.com/Polymer/tools/tree/master/packages/polyserve) to [@web/dev-server](https://modern-web.dev/docs/dev-server/overview/).

Is it possible to show one route over another?

7

My use case is: I have a `page-user` element for the route `/user/:id`. In this element, I request the user info in the `onBeforeEnter` callback using the `location.params.id`. When the...

The polymer-cli is used for the `docs:analyze` script: ``` polymer analyze dist/vaadin-router.js src/documentation/*.js > analysis.json ``` Maybe we can drop it in favor of [runem/web-component-analyzer](https://github.com/runem/web-component-analyzer) and [web-padawan/api-viewer-element](https://github.com/web-padawan/api-viewer-element)? Related with: https://github.com/vaadin/vaadin-router/issues/363

Access to this.location: Property 'location' does not exist

6

The docs says that we can access to `this.location` to get the route parameters: https://vaadin.github.io/vaadin-router/vaadin-router/#/classes/Router/demos/demo/index.html And it works. But when using TypeScript, I get an error: >Property 'location' does not...

Option to remove the trailing slash

3

I would like to have the option to remove the trailing slash, to avoid the SEO duplicate content issue. When a user go to: `myapp.com/user/abdonrd/` Redirects to: `myapp.com/user/abdonrd`

Ignore auto generated files

1

What do you think about ignore the generated files like `docs`, `analysis.json`, etc? Easier to see the PRs.

[control-state-mixin] Rename focus-ring to focus-visible

1

I know it would be a breaking change, but... Would it make sense to rename `focus-ring` to `focus-visible`? To be closer to the specification: https://developer.mozilla.org/en-US/docs/Web/CSS/:focus-visible https://github.com/vaadin/component-mixins/blob/c83bb8903ef1a84e011035b3ba30a99dfbc98e7f/packages/focus-visible-mixin/focus-visible-mixin.ts#L76-L82

Currently it is the grading API that handles the `/sync` endpoint. We should move this to the new Challenges API. And so the grading only takes care of grading.