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

[Bug]: Wrong <input> type for url #550

Closed
Lapotor opened this issue May 27, 2024 · 3 comments · Fixed by #556
Closed

[Bug]: Wrong <input> type for url #550

Lapotor opened this issue May 27, 2024 · 3 comments · Fixed by #556

Comments

@Lapotor
Copy link
Contributor

Lapotor commented May 27, 2024

What happened?

On popups where you have a URL field the <input> type is set to type="href" this is not a valid <input> type.

It should be type="url" to match valid valid <input> type.

<input type="href" [id]="getId('link-popup-url')" formControlName="href" autocomplete="off" />

<input type="href" [id]="getId('image-popup-url')" formControlName="src" autocomplete="off" />

Version

v17.5.4

Angular Version

v17.3.9

What browsers are you seeing the problem on?

Firefox

Link to reproduce

No response

Relevant log output

No response

Willing to submit a PR?

Yes

@Lapotor
Copy link
Contributor Author

Lapotor commented May 27, 2024

There for this rule don't apply to the URL fields:

input[type='text'],
input[type='url'] {
padding: 2px 4px;
}

@sibiraj-s
Copy link
Owner

Thanks for catching this, Please raise a PR. will get that deployed soon.

Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in the thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants