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

Eatyourpeas/issue96 #104

Merged
merged 2 commits into from
Jun 6, 2024
Merged

Eatyourpeas/issue96 #104

merged 2 commits into from
Jun 6, 2024

Conversation

eatyourpeas
Copy link
Member

Overview

Adds spinners to htmx request radio buttons and csv upload button

Code changes

Add daisyui spinner and css classes to show/hide + hyperscript

Related Issues

closes #96 #64

Mentions

@mentions of the person or team responsible for reviewing proposed changes.

@eatyourpeas eatyourpeas merged commit 594bf1f into development Jun 6, 2024
2 checks passed
@eatyourpeas eatyourpeas deleted the eatyourpeas/issue96 branch June 6, 2024 19:05
@mbarton
Copy link
Member

mbarton commented Jun 7, 2024

@eatyourpeas please could you add a more descriptive title to your PRs? :)

Even just copying the first line of the description. It really helps anyone who is looking at the GitHub notification stream or is going back through git blame or logs to investigate why something was added

@eatyourpeas
Copy link
Member Author

I am so sorry yes of course - you have asked me to do this before. my bad. It is largely because I use VS Code for the PR workflow which prepopulates the title field and I am so focussed on writing a detailed PR that I forget the title. Apologies and will do better :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

spinners to view preference buttons
2 participants