[x] Reference the open issue that the pull request addresses
[x] Pass code quality checks
spin up docker docker-compose up -d --build
enter api container docker-compose exec api /bin/bash
run api tests make validate
exit container ctrl/command+D or exit
enter web container docker-compose exec web /bin/sh
run front-end tests npm run test or npx jest
lint npm run lint-fix
exit container as above
[x] Request code review
Please allow 36 hours from opening a pull request before merging a pull request- even if it has already received an approving review.
[ ] Address comments on code and resolve requested changes
[ ] Merge own code
Description
Issue: #437
Brief description of solution
In the function that disables each date if it doesn't have data, also disable the currently selected date
Fix style priorities by adding :disabled selector before styles for the currently selected date
Test that clicking the currently selected date does nothing
Note: both this and the fix for 443 modify menu.css, so maybe we should finish and merge the first pull request first so that I can check that this one doesn't break.
Checklist
docker-compose up -d --build
docker-compose exec api /bin/bash
make validate
ctrl/command+D
orexit
docker-compose exec web /bin/sh
npm run test
ornpx jest
npm run lint-fix
Description
Issue: #437
Brief description of solution
:disabled
selector before styles for the currently selected dateNote: both this and the fix for 443 modify menu.css, so maybe we should finish and merge the first pull request first so that I can check that this one doesn't break.