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

Support to search for a senator #1368

Closed
wants to merge 4 commits into from

Conversation

MutazAshhab
Copy link
Contributor

This PR is a revised version of PR #1329 as that PR had issues with the regression tests as well as an ugly commit history. It relates to issue: #1198

The placeholder value rotates between the randomly generated suggestion as well as e.g. Senate for Tasmania and e.g. Senate for NSW to avoid the placeholder being wider than the width of the input box

Here is what it looks like:

UI_changes

The search supports all ways of representing a constituency e.g. Queensland or QLD

In the case that a user has javascript disabled then the placeholder values wont rotate and the user will not know that the

@stale
Copy link

stale bot commented Sep 4, 2022

This issue has been automatically marked as stale because there has been no activity on it for about six months. If you want to keep it open please make a comment and explain why this issue is still relevant. Otherwise it will be automatically closed in a week. Thank you!

@stale stale bot added the wontfix label Sep 4, 2022
@stale stale bot closed this Sep 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant