-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Date picker modal for /tag/__ pages #11199
Conversation
Code Climate has analyzed commit be16915 and detected 0 issues on this pull request. View more on Code Climate. |
Hi @jywarren @TildaDares @cesswairimu, |
This pull request generated screenshots of many common pages in the running app. You should be able to download and view them here: |
Hi! Excited to see this! Would you mind posting a screenshot in a comment?
Thank you!
…On Sat, Jun 18, 2022, 2:19 AM github-actions[bot] ***@***.***> wrote:
This pull request generated screenshots of many common pages in the
running app. You should be able to download and view them here:
https://github.com/publiclab/plots2/suites/6984092632/artifacts/273269598
—
Reply to this email directly, view it on GitHub
<#11199 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAF6J6RFK7HUOG7WOPREADVPTMUVANCNFSM5ZDJKXBA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Ah sorry, my connection isn't great and I've been following by email. I'll
check next time, thanks!
…On Sat, Jun 18, 2022, 5:12 PM Karishma Vanwari ***@***.***> wrote:
Sure. I've already added it in the PR description too.
[image: image]
<https://user-images.githubusercontent.com/78212650/174435131-fb93307c-bc47-438a-a506-634c887bb404.png>
—
Reply to this email directly, view it on GitHub
<#11199 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAF6J7NIQEYHA5UVHHLVN3VPWVJXANCNFSM5ZDJKXBA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks super. Does it render well on narrow screens like mobile phones? Thanks!
Yes, it does, @jywarren! I hope this works: |
Excellent, thank you @KarishmaVanwari!!! |
Thank you @jywarren!! |
Added the date picker modal (UI) for /tag/__ pages that opens on clicking the "custom" option in the "Sort By" dropdown.
Fixes #11198
Screenshot:
![image](https://user-images.githubusercontent.com/78212650/174394604-215a9f88-43e7-4d33-a9ea-b0887c73d6d1.png)
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test
@publiclab/reviewers
for help, in a comment below