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

Readonly inputs shouldn't be grayed out #33925

Closed
tkrotoff opened this issue May 10, 2021 · 5 comments · Fixed by #33961 or #36499
Closed

Readonly inputs shouldn't be grayed out #33925

tkrotoff opened this issue May 10, 2021 · 5 comments · Fixed by #33961 or #36499
Labels

Comments

@tkrotoff
Copy link
Contributor

tkrotoff commented May 10, 2021

Bootstrap styles readonly inputs like disabled inputs (since v1.3.0 FYI):

image

However browsers don't do that:

Chrome 90

readonly:

image

vs disabled:

image

IE 11

readonly:

image

vs disabled:

image

Firefox 88

readonly:

image

vs disabled:

image

Safari 13.2.1

readonly:

image

vs disabled:

image

Edge 90

readonly:

image

vs disabled:

image

GNOME Web 3.36.4

readonly:

image

vs disabled:

image

Frameworks that don't style readonly:

(when there is no demo for readonly/readOnly/isReadOnly, I've manually modified another demo to check behavior)

Brands:

Frameworks that style readonly:




Considering all this, I suggest that Bootstrap readonly input should not be styled as disabled input.

@alecpl
Copy link
Contributor

alecpl commented May 11, 2021

Related issue #33860.

@alecpl
Copy link
Contributor

alecpl commented May 13, 2021

@mdo I think this ticket was a feature request "Remove readonly state styles completely from Bootstrap". So, it deserves a comment. Could you re-consider this? It would not happen before v6 anyway, I guess, but was that ever considered?

Edit: Sorry, I wrote this comment thinking the ticket has been closed. That's why I used past tense.

@tkrotoff
Copy link
Contributor Author

@XhmikosR this feature request is not related to #33961 and should not be closed because of it. Please re-read.

@XhmikosR
Copy link
Member

@mdo had linked this in #33961, thus it was automatically closed.

@XhmikosR XhmikosR reopened this May 13, 2021
@mdo mdo moved this to To do in v5.2.0-stable May 20, 2022
Repository owner moved this from To do to Done in v5.2.0-stable Jun 14, 2022
@mdo mdo moved this to Done in v5.3.0 Jul 19, 2022
@mdo mdo added this to v5.3.0 Jul 19, 2022
@mdo mdo removed this from v5.3.0 Oct 3, 2022
@tkrotoff
Copy link
Contributor Author

@mdo can you re-open this ticket for v6?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Done
5 participants