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

disable post button on successful post / disable focus on that button #7

Open
adri1s opened this issue Nov 20, 2021 · 3 comments
Open
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@adri1s
Copy link
Member

adri1s commented Nov 20, 2021

disable the post button after with post and delete the text below.
Put a ✔ instead of the post button for example

@adri1s adri1s added the enhancement New feature or request label Nov 20, 2021
@gummyWalrus
Copy link
Collaborator

gummyWalrus commented Nov 23, 2021

Issue solved in commit 8a2b09
The button has been deleted and post success is now notified by a green 'check' replacing the loading animated icon.

@adri1s
Copy link
Member Author

adri1s commented Nov 25, 2021

KO on 0.1.9

@adri1s adri1s reopened this Nov 25, 2021
@gummyWalrus
Copy link
Collaborator

Issue fixed in commit 8a2b09 and the button is not reposting since but commit 841dbe sets the cursor as 'default' when hovering the button after a successful post to make it clear that it is not clickable.

Still looking for a way to prevent focus on click thought. If anyone has a solution he's welcome to comment it or implement it via a pull request

@gummyWalrus gummyWalrus added the good first issue Good for newcomers label Nov 29, 2021
@gummyWalrus gummyWalrus changed the title disable post buttom after post disable post button on successful post / disable focus on that button Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants