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

Update bug_report.md less is more #2323

Merged
merged 3 commits into from
May 31, 2024
Merged

Conversation

raszpl
Copy link
Contributor

@raszpl raszpl commented May 31, 2024

Less text, more concise.

@ImprovedTube ImprovedTube merged commit bdd7fcb into code-charity:master May 31, 2024
@raszpl raszpl deleted the patch-5 branch May 31, 2024 20:39
@ImprovedTube
Copy link
Member

hi! @raszpl
these can stay a bit verbose for the average user to understand:

  • How to reproduce this bug? Might only happen with a certain setting of our's?
  • Are any errors shown in the Browser-Console? (F12-key, then console-tab)

this is a bit long:

  • ImprovedTube Version (Extension popup after closing Search, chrome:extensions, about:addons):

those (fewer) people who know chrome://extensions, might also know how to see the version there already
, while most of people might know/remember how the stores look like, and find the version number there sooner.
(OS or Device aren't impossible to matter either.)

@ImprovedTube
Copy link
Member

*...Concluding, you might be motivated, when everything is criticized , such as 2293 (if not both of you or protective Vivaldi fans), Isn't such motivation stressful though? (if not, i should i might start to publish frustrations about almost everything 😲)(More seriously, to "protect oneself from trolls", we can care equally for all issues or the thumbs for the release)

@raszpl
Copy link
Contributor Author

raszpl commented May 31, 2024

  • How to reproduce this bug? Might only happen with a certain setting of our's?

"Steps to reproduce" is enough. Keep it simple. Settings dont really matter at the moment of bug submission. Can ask for settings later when unable to reproduce.

  • Are any errors shown in the Browser-Console? (F12-key, then console-tab)

Its not F12 in all browsers\oses. I specifically left it brief as a filter - If someone doesn know what devtools/console is he wont be able to recognize what is a valid error message and what is not.

this is long and not complete:

  • ImprovedTube Version (Extension popup after closing Search, chrome:extensions, about:addons):

(most people might know/remember how the stores look like. those who know chrome://extensions, might also know how to see the version there already)

Its as long as the text you replaced it with, except:

"(Written in the Extension header & your Browser's store)"

version is not showing in extension header, you need to close search
version is not showing anywhere in Browser store? https://chromewebstore.google.com/detail/improve-youtube-%F0%9F%8E%A7-for-yo/bnomihfieiccainjcjblhegjgglakjdd

In fact in newest Chrome even chrome:extensions is not enough, its still hidden under Details :/

What I didnt like in the first place :

  • Description: (Clear/concise. Screenshot/Video welcome)

Whats the point of telling someone to be clear and concise? Its patronizing. Moving instructions to the right side of colon means people will just live whole line like this, cant expect people to edit the template.

  • Your Settings (From the Extension's -Hamburger menu > Settings > Backup & reset > Export settings (You can remove any Blocklist, History Analyzer data - or paste the following in the Browser-Console and hit enter: (function({analyzer, blocklist, ...a}){return JSON.stringify(a)})(ImprovedTube.storage) )

trailing ')'. Now that I clicked New Issue again I realize template doesnt display formatted, so the whole code line will give most non technical people grief, should be in new line to avoid confusion

  • Your Settings (From the Extension's -Hamburger menu > Settings > Backup & reset > Export settings (You can remove any Blocklist, History Analyzer data - or paste the following in the Browser-Console and hit enter:
    (function({analyzer, blocklist, ...a}){return JSON.stringify(a)})(ImprovedTube.storage)

  • YouTube-Document: (F12 copy the whole root element: <html> (alternative: CTRL+U) (https://github.com/code-charity/youtube/wiki/Contributing#Test )

How many people know what a root element is? :) I wouldnt even know how would that be of any use to me while debugging. Its one of those things that can be asked later if its actually needed.

  • OS / Device:

as above no idea how that would help me with debugging.

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.

None yet

2 participants