Skip to content

feat(eslint-plugin-template): [i18n] add additional default ignores #2201

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rgant
Copy link

@rgant rgant commented Jan 14, 2025

Fix for #997

Found some issue with the current issue request:

  1. I don't believe displayMode is an @Input for mat-accordion in Angular Material 19 any longer.
  2. Adding non-void tags to ignore tags ignores all the attributes and text contents of those tags. Which is not always ideal.
    1. mat-drawer can just have text content. https://material.angular.io/components/sidenav/examples#sidenav-backdrop
    2. mat-icon can have aria-labels. https://material.angular.io/components/icon/examples#icon-overview
    3. ng-container is specifically recommended by Angular documentation for i18n. https://angular.dev/guide/i18n/prepare#translate-inline-text-without-html-element
    4. title is used in SVG and can be translated using i18n.
    5. :svg:polygon I am unaware of what is special about the <polygon> element that needs to be ignored.

@rgant
Copy link
Author

rgant commented Jan 14, 2025

Also, the documentation links for the i18n docs on angular.dev are wrong. Easiest for me is to just correct that in the PR. However if you prefer separate requests then I will try to find time to do that.

@rgant
Copy link
Author

rgant commented Jan 14, 2025

Also, I couldn't find developer setup documentation, nor contributing documentation. So I ran the update-rule-docs manually. Not sure if that was desired.

@rgant
Copy link
Author

rgant commented Jan 14, 2025

All I care about is NgOptimizedImage but I found this (too?) large issue and decided to try to resolve as much of it as I could. If there are any concerns then I can simplify this down to only the image changes easily.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant