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

Add Git FAQ icon to start menu linking to the wiki/FAQ #283

Merged
merged 1 commit into from
May 3, 2020

Conversation

PhilipOakley
Copy link
Contributor

Signed-off-by: Philip Oakley philipoakley@iee.email

Note that the icons are listed in alphabetic order on W10 start menu (on my machine), so we have:
Bash, CMD, FAQs, GUI, Release Notes.

@dscho
Copy link
Member

dscho commented May 2, 2020

Does that really work if you use a URL for a filename?

@PhilipOakley
Copy link
Contributor Author

Does that really work if you use a URL for a filename?

Yes it does. I was concerned too. Note that the {app} is dropped. But mainly see Innosetup [Icons] section where it says:

In addition to file and folder names, URLs (web site addresses) may also be specified. When a URL is specified, Setup will create an "Internet Shortcut" (.url) file, and ignore the Parameters, WorkingDir, HotKey, and Comment parameters.

The examples then shows the dropping of the {app}.

I now see that I could have dropped the Parameters, WorkingDir, HotKey, and Comment parameters. 😉

From the Innosetup [Icons] section
https://jrsoftware.org/ishelp/index.php?topic=iconssection

"In addition to file and folder names, URLs (web site addresses)
may also be specified. When a URL is specified, Setup will create
an "Internet Shortcut" (.url) file, and ignore the Parameters,
WorkingDir, HotKey, and Comment parameters."

Their example also shows the '{app}' path portion also being dropped.

Signed-off-by: Philip Oakley <philipoakley@iee.email>
@PhilipOakley
Copy link
Contributor Author

Updated the commit message, and tweaked the changes to drop the no longer required parameters for the Icon setup.

@dscho dscho merged commit ac22c32 into git-for-windows:master May 3, 2020
@dscho
Copy link
Member

dscho commented May 3, 2020

Thanks!

dscho added a commit that referenced this pull request May 3, 2020
The Frequently Asked Questions (FAQ) [are now linked in a Start Menu
item](#283).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
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.

2 participants