-
Notifications
You must be signed in to change notification settings - Fork 525
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
408a992
commit f85eb9d
Showing
3 changed files
with
13 additions
and
7 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,17 +1,18 @@ | ||
--- | ||
name: Tools | ||
about: Report issues with the updaters, troubleshooter, or any other tools. | ||
about: Report issues with the updaters, the troubleshooter, or any other tools. | ||
title: '' | ||
labels: '' | ||
assignees: '' | ||
|
||
--- | ||
|
||
<!-- | ||
Make sure to specify: | ||
Please specify: | ||
- the name of the file (i.e. updater.bat, updater.sh) | ||
- the OS where you used the tool (if applicable) | ||
- steps to reproduce the issue | ||
- expected result | ||
- actual result | ||
- expected result | ||
Blank out this field before typing, or start typing after the next line. | ||
--> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
f85eb9d
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.
👖, when you get down to tweaking these, another thing that you might want to mention in
user-js.md
, is that you don't want people throwing massive lists of prefs at you (#442, #542, etc).f85eb9d
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.
thanks 🐈 but that is so rare now, that I'd rather leave the instructions as short as possible
f85eb9d
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.
That's fine, but I think you're free to go wild here, considering we can make the templates as specific as we want. The way I wrote these templates, they're not really templates. They're more like guidelines, meant to filter out stuff. The idea is that users choose the one that seems to match their intentions more, and when they do, they get a bunch of "uh, yeah, you wanna do that? Great. But first, do this and this, and be aware that we don't want you to do this"
Also, notice my choice of words... Suggest changes to user.js. If you wanted, you could use the same strategy for more extreme scenarios, for example, you could add a separate "template" saying Demand changes to user.js in the description, that when selected by the user just says Nuh-uh, you're not entitled to demanding shit (see the license). Fuck off