Pin Topic Buttons Inactive?


(Aaron Makaruk) #1

When trying to pin a topic, the buttons aren’t active - see below:


(cpradio) #2

Select a Date, then they’ll become active


(Jeff Atwood) #3

(Jeff Atwood) #4

Note to @zogstrip it might be better to pop a descriptive error here rather than just disabling buttons.


(Régis Hanol) #5

You mean change the text of the disabled buttons to something like “select a date to pin this topic”?


(Jeff Atwood) #6

No, on click do a red bouncy error like we do in the editor. Bouncing over the missing field…


(Régis Hanol) #7

This commit adds a red popup with a descriptive message when the date is missing :fish:

https://github.com/discourse/discourse/commit/05adcda1fc0ef3422048c66a3292c688da7667bb