- Issue created by @srishtiiee
- last update
over 1 year ago 30,057 pass - @srishtiiee opened merge request.
- last update
over 1 year ago 30,057 pass - Status changed to Needs review
over 1 year ago 8:33am 24 August 2023 - Status changed to Needs work
over 1 year ago 9:05am 24 August 2023 - last update
over 1 year ago 30,055 pass, 1 fail - Status changed to Needs review
over 1 year ago 10:00am 24 August 2023 - last update
over 1 year ago 30,055 pass, 1 fail - Status changed to RTBC
over 1 year ago 11:08am 24 August 2023 - 🇩🇪Germany marcoliver Neuss, NRW, Germany
Works fine! The test fail seems unrelated.
Marking as RTBC.
- last update
over 1 year ago 30,058 pass - last update
over 1 year ago 30,060 pass - last update
over 1 year ago 30,060 pass - last update
over 1 year ago 30,098 pass - 🇪🇸Spain ckrina Barcelona
Setting the button as disabled in this situation is an improvement, already as it is. So +1 for getting this in.
But it would be great if we could come up with a pattern that gives more context/feedback to the user about the "why" that button is disabled. My first thought was using the
title
attribute with something like "Mandatory value not removable". Buttitle
won't show up on hover if the button is disabled, plus I'm not sure it's a right use of it. Maybe, as @lauriii suggested, this could be a nice use of the Tooltip once that's in? - Status changed to Fixed
over 1 year ago 4:49pm 31 August 2023 Automatically closed - issue fixed for 2 weeks with no activity.