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

hide non-existent stats from non-cnt users #226

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

michaelkirk
Copy link
Contributor

before:

Screenshot 2025-03-12 at 13 52 29

after:

Screenshot 2025-03-12 at 13 52 21

@@ -222,8 +222,7 @@
<div style="border: dashed black 2px; padding: 8px">
{#if selectedBoundary}
<div style="display: flex; justify-content: space-between;">
<b>Your Neighbourhood Overall</b>
<!--REVIEW: good place for a name input field? -->
<b>Your neighbourhood overall</b>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

iOS has a weird convention where you TitleCase all headings. I'm trying to unlearn it for web, but often forget. Feel free to correct me if you notice!

@michaelkirk michaelkirk force-pushed the mkirk/cnt-ltn-stats branch from 575c62b to ed97c72 Compare March 12, 2025 23:49
@michaelkirk michaelkirk merged commit ed97c72 into a-b-street:main Mar 12, 2025
1 check passed
@michaelkirk
Copy link
Contributor Author

I'm going to merge this because it's fairly trivial.

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.

1 participant