[3.x] Disable checkout button and add notice if not in stock #128
Conversation
BobWez98
requested review from
JimmyHoenderdaal,
Roene-JustBetter and
jordythevulder
| </x-rapidez-ct::button.enhanced> | ||
|
|
||
| <div class="w-full" :class="{ 'cursor-not-allowed': !canOrder }"> | ||
| <x-rapidez-ct::button.enhanced |
There was a problem hiding this comment.
isn't the disabled attribute not better suited for this instead of pointer-events-none?
There was a problem hiding this comment.
As this button has a href, this is a <a href and not a button. The disabled attribute does not work for a link.
There was a problem hiding this comment.
And what if we replaced the "continue" button with a button that removes the "Out of stock" products. "Remove out of stock products" for example. Or doesn't that functionality cover the scope of this user story?
There was a problem hiding this comment.
Good idea, create a story for it.
|
Seems like this requires something from the core that isn't merged yet? This package should require the correct core version when that's merged and released. |
|
This has been reverted, please arrange a follow-up with the right version constraint |
5 participants
Also see depending on:
rapidez/core#627
rapidez/core#628
Internally there's a story to have some extra frontend added for this.
V2: #127