Skip to content
This repository was archived by the owner on Sep 13, 2022. It is now read-only.

Helpbutton#100

Open
thermoscookies wants to merge 7 commits into
Glennmen:developfrom
thermoscookies:helpbutton
Open

Helpbutton#100
thermoscookies wants to merge 7 commits into
Glennmen:developfrom
thermoscookies:helpbutton

Conversation

@thermoscookies

Copy link
Copy Markdown

Designed to give us an option to create a help page or video for users to help explain how the map works.

Comment thread pre-index.php
title="PayPal - The safer, easier way to pay online!" alt="Donate" style="float: right;">
</a>';
}
if ($helplink != "") {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change this to if (!empty($helplink)) {

Comment thread pre-index.php
</a>';
}
if ($helplink != "") {
echo '<a href="' . $helplink . '" target="_blank" style="margin-bottom: 5px; vertical-align: middle; padding:0 5px;">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move CSS style into one of the SCSS files.

@thermoscookies thermoscookies Feb 23, 2018

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@iroken222 I copied and pasted this from the paypalurl/discordurl above it, do you want me to move those into stylesheets as well so they can be consistent?

Comment thread pre-index.php
if ($helplink != "") {
echo '<a href="' . $helplink . '" target="_blank" style="margin-bottom: 5px; vertical-align: middle; padding:0 5px;">
<img src="static/images/help.png" border="0" name="submit"
title="Help for this map." alt="Help" style="float: right;">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move styles into stylesheets.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants