Exclude disabled features from "Find feature" steps
See original GitHub issueDescription
I was using the awesome “Find Feature” button to figure out why in my list of PRs and issues some are grey.
It explained to me that the issue was coming form dim-bots
, but when I checked, it was already disabled. Luckily I discovered dim-visited-conversations
along the way since I only searched for dim
, so I was abe to disable it.
But I wonder why the it even suggests features that are disabled…
I think it could also be considered a bug, but the form for that didn’t seem to fit.
Steps to reproduce:
- Disable
dim-bots
and enabledim-visited-conversations
- Open any list of PRs or issues where you can see some title being dimmed and others aren’t (the link I added works for me but might not work for you)
- Open the options, scroll to the bottom and click “Find Feature”
- go back to the tab from step 2 and refresh the page
- click “yes” whenever some PR is dimmed and “no” if all PRs are black/white
Screenshot
No response
Example URL
Issue Analytics
- State:
- Created 2 years ago
- Comments:9 (5 by maintainers)
Top Results From Across the Web
How do I disable a feature in specflow (Gherkin) without ...
I have found that the @ignore attribute needs to be added to every single Scenario within a feature. – DanH. May 25, 2017...
Read more >Remove a device from Find Devices on iCloud.com
You can use Find Devices on iCloud.com to temporarily remove a device from the Devices ... Turning off iCloud turns off all iCloud...
Read more >Remove hidden data and personal information by inspecting ...
Find and remove hidden data and personal information with the Document Inspector · Open the Word document that you want to inspect for...
Read more >Feature flags - GitLab Docs
Edit the feature flag you want to remove. Select More actions ( ). Select Search code references. Disable a feature flag for a...
Read more >Remove Hidden Data in Microsoft Excel - Marquette University
Review the document inspection results. After you have completed these steps, save the document, and the spreadsheet will not contain the hidden data...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Thank you for the recording! I think I understand where the issue is coming from: on the fourth step (with 6 steps remaining), you clicked “Yes” where it should have been “No”, because you saw the collapsed issue at the bottom of the page.
This is not caused by
dim-visited-conversations
, butdim-bots
instead, which collapses and dims every PR, issue and commit made by a bot. And the “Find feature” mechanism only works properly if focusing on the effects of a single feature.Your point is valid nonetheless, because the feature finder always starts with every feature enabled, regardless of the user’s settings. Since the user might be exposed to features they aren’t used to seeing, this can cause confusion and derail the process.
So I think we should make the finder work with only the features already enabled. Not only would it be more straightforward, it’d be faster too (less features to consider = less steps).
Thanks for the detailed report!
However I tried to reproduce it in the same conditions as mentioned, and I got the correct answer of
dim-visited-conversations
.My guess is that you must have clicked the wrong button during one of the steps — probably because the color of the visited links isn’t very different from the regular one.