Alya Abbott
Alya Abbott
A disabled button should look like other disabled buttons, e.g., "Confirm" in the move messages modal:
Is this PR is WIP (like the title says), or is it ready for review?
You also need to complete the self-review process first.
> I guess disabled button color is now changed to blue, so therefore I have also implement same thing create channel button Sounds good; please make sure the screenshots in...
@sahil839 Does it make sense for you to take a look at this stage?
Testing the PR, in channels where the current user isn't allowed to start a topic: 1. Let just hide the `+` button in the sidebar rather than showing a disabled...
Playing with the PR: 1. I think it would be good to change the tooltips on the `+` left sidebar button and "Start new conversation" button to say "New message"...
I noticed that the "Schedule message" button doesn't seem to do anything when you try to use it in the state where you can't send to the current topic. But...
3. You get different error feedback depending on whether you try to resolve from the message header bar (error tooltip), left sidebar topic menu (banner using an outdated design), or...
That's all I've got from this round of testing! I was focusing on the UX, not on validating that different configurations work expected.