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

Replace "Page wiki" button with new "Report a bug" button #1013

Merged

Conversation

CyberShadow
Copy link
Member

@@ -211,6 +211,12 @@ _=
PAGE_TOOLS=
$(DIVID tools,
$(SPANC tip,
<a href="https://issues.dlang.org/enter_bug.cgi?bug_file_loc=http%3A%2F%2Fdlang.org/$(SELF_PATH)&bug_severity=enhancement&component=$(BUGZILLA_COMPONENT)&op_sys=All&priority=P3&product=D&rep_platform=All&short_desc=%5B$(TITLE)%5D&version=D2" class="button">Report a bug</a>
Copy link
Member

Choose a reason for hiding this comment

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

$(HTMLTAG3 a, href="..." class="button", Report a bug)

Copy link
Member Author

Choose a reason for hiding this comment

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

Eh, done, but I don't see the point frankly. It's longer and carries no semantic meaning (there is no meaningful alternative value for HTMLTAG3 one may want to use)..

Copy link
Member Author

Choose a reason for hiding this comment

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

Plus it's now inconsistent with the other button's HTML code. Such changes are probably better done separately.

@CyberShadow CyberShadow force-pushed the pull-20150609-004423-reportbugbtn branch from 6506ce4 to 6521402 Compare June 9, 2015 04:41
@@ -1,5 +1,7 @@
ROOT_DIR =
ROOT = .
SELF_PATH =
BUGZILLA_COMPONENT = websites
Copy link
Member

Choose a reason for hiding this comment

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

Oops, these just changed, please replace: http://beta.forum.dlang.org/post/ml5tr4$1na4$1@digitalmars.com

Copy link
Member Author

Choose a reason for hiding this comment

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

Done - turns out we don't need BUGZILLA_COMPONENT now, because we already have PROJECT :)

@CyberShadow CyberShadow force-pushed the pull-20150609-004423-reportbugbtn branch from 6521402 to 138b471 Compare June 9, 2015 13:28
andralex added a commit that referenced this pull request Jun 9, 2015
…bugbtn

Replace "Page wiki" button with new "Report a bug" button
@andralex andralex merged commit 33a7b8a into dlang:master Jun 9, 2015
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