-
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
empty wiki table header #7542
empty wiki table header #7542
Conversation
Codecov Report
@@ Coverage Diff @@
## main #7542 +/- ##
==========================================
+ Coverage 81.87% 82.01% +0.13%
==========================================
Files 97 101 +4
Lines 5612 5894 +282
==========================================
+ Hits 4595 4834 +239
- Misses 1017 1060 +43
|
You could hide buttons "by type" & "Sort by", as well as "Related topics." |
Screenshots 📸 (click to expand)7542-test_questions.png7542-test_embeddable_grids.png7542-test_signup.png7542-test_viewing_the_settings_page.png7542-test_tag_by_author_page.png7542-test_wiki_page_with_inline_grids.png7542-test_stats.png7542-test_viewing_the_dashboard.png7542-test_searching_an_item_from_the_homepage.png7542-test_signup_modal_form_validation.png7542-test_tag_stats.png7542-test_login_modal_form_validation.png7542-test_questions_shadow.png7542-test_login_modal.png7542-test_profile_page.png7542-test_comments.png7542-test_tags.png7542-test_signup_modal.png7542-test_wiki.png7542-test_methods.png7542-test_tag_page.png7542-test_blog_page_with_location_modal.png7542-test_tag_wildcard.png7542-test_signup_modal_disabled_submit_button_on_empty_username.png7542-test_embeddable_thumbnail_grids.png7542-test_front_page_with_navbar_search_autocomplete.png7542-test_spam_moderation_page.png7542-test_login.png7542-test_viewing_the_dropdown_menu.png7542-test_viewing_question_post.png7542-test_mobile_displays.png7542-test_simple-data-grapher_powertag.png7542-test_front.png7542-test_question_page.png7542-test_tag_contributors_page.png7542-test_blog.png7542-test_people.png7542-test_wiki_revisions.pngLearn about automated screenshots Generated by 🚫 Danger |
"By type" drop down is required for switching type thatswhy it is important but sort by and related topics can be hidden |
The "by type" button is used for filtering the results... if there are no results then you can't filter anything. So it can be safely hidden as well. |
@nstjean you are right, I will do this |
Great collab going here too |
Hi @keshavsethi, maybe I am missing something but I cannot see the message added from the screenshot |
@keshavsethi please reply to @cesswairimu |
Hey @cesswairimu @SidharthBansal, Sorry for the late reply. I just reviewed it. The message is coming when wiki table is empty. Can you please share your screenshot or give me more details of whatever you are getting.
Thnaks!! |
Hi 😄, this issue has been automatically marked as stale because it has not had recent activity. Don't worry you can continue to work on this and ask @publiclab/reviewers to add "work in progress" label 🎉 . Otherwise, it will be closed if no further activity occurs in 5 days -- but you can always re-open it if you like! 💯 Thank you for your contributions 🙌 🎈. |
Trying to re-trigger tests!! |
Nice, sorry this took so long!!! |
* wiki table header * empty table header
* wiki table header * empty table header
* wiki table header * empty table header
* wiki table header * empty table header
* wiki table header * empty table header
* wiki table header * empty table header
* wiki table header * empty table header
* wiki table header * empty table header
Fixes #7531
message added when wiki table is empty
@VladimirMikulic @cesswairimu @nstjean @SidharthBansal
Thanks!