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

Updated HICSS style #2920

Merged
merged 2 commits into from Sep 3, 2017

Conversation

@GeorgLink
Copy link
Contributor

commented Sep 3, 2017

The style matches the template better:
http://hicss.hawaii.edu/wp-content/uploads/2016/09/hicss-formatspec-initialmanuscript.doc

Known issue:

  • The comma after the title should be outside the quotation marks.
@csl-bot

This comment has been minimized.

Copy link

commented Sep 3, 2017

Awesome! You've created a pull request to the Citation Styles Language styles repository. We'll get in touch soon (usually within a day or two). In the meantime, our automated test system will go ahead and run some checks on your pull request. In a few minutes you'll be notified of the test results.

If you haven't done so yet, please make sure your style validates and follows all our other Style Requirements.

To update the current pull request, visit the "Files changed" tab above, and click on the pencil icon (see below) in the top-right corner of your style to start editing.

image

If you need assistance at any point, please leave a comment and we'll get back to you (feel free to write in Dutch, English, French, German, Portuguese, or Spanish).

@csl-bot

This comment has been minimized.

Copy link

commented Sep 3, 2017

😃 Your submission passed all our automated tests.

@GeorgLink

This comment has been minimized.

Copy link
Contributor

commented on d8f040f Sep 3, 2017

Thanks @adam3smith

@csl-bot

This comment has been minimized.

Copy link

commented Sep 3, 2017

😃 Your submission passed all our automated tests.

@adam3smith
Copy link
Member

left a comment

A couple of quick questions

@@ -36,10 +44,10 @@
<macro name="title">
<choose>
<if type="bill book graphic legal_case legislation motion_picture report song" match="any">
<text variable="title" font-style="italic" quotes="false"/>
<text variable="title" quotes="false" font-style="italic"/>

This comment has been minimized.

Copy link
@adam3smith

adam3smith Sep 3, 2017

Member

why the italics? Not seeing them in the styleguide.

This comment has been minimized.

Copy link
@GeorgLink

GeorgLink Sep 3, 2017

Author Contributor

same as above

<names variable="author">
<name initialize-with="." delimiter=", " and="text" name-as-sort-order="all" sort-separator=", "/>
<name and="text" delimiter-precedes-et-al="always" delimiter-precedes-last="always" initialize-with="." name-as-sort-order="first"/>

This comment has been minimized.

Copy link
@adam3smith

adam3smith Sep 3, 2017

Member

Do we have anything more than the template? One of the examples corresponds to name-as-sort-order="first", the other ="never"

This comment has been minimized.

Copy link
@GeorgLink

GeorgLink Sep 3, 2017

Author Contributor

Unfortunately, HICSS does not have a good style guide and when I pull examples from last year the authors follow different conventions and get published. The option reflects what I believe to look the best, based on previous years, while still adhering to the limited style guide.

This comment has been minimized.

Copy link
@GeorgLink

GeorgLink Sep 3, 2017

Author Contributor

PS: I have a conditional accept with these settings and no comment was made about changing the style.

<text variable="container-title"/>
<text variable="volume"/>
<group font-style="normal" suffix=", ">
<text variable="container-title" font-style="italic"/>

This comment has been minimized.

Copy link
@adam3smith

adam3smith Sep 3, 2017

Member

Same here -- any indication of italics somewhere other than the styleguide?

This comment has been minimized.

Copy link
@GeorgLink

GeorgLink Sep 3, 2017

Author Contributor

same as above

@adam3smith adam3smith merged commit 3b2498f into citation-style-language:master Sep 3, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@adam3smith

This comment has been minimized.

Copy link
Member

commented Sep 3, 2017

OK, let's go with this then. Thanks!

@GeorgLink

This comment has been minimized.

Copy link
Contributor Author

commented Sep 3, 2017

Thank you!

@GeorgLink

This comment has been minimized.

Copy link
Contributor Author

commented Sep 12, 2017

Update: The HICSS paper was accepted with this style. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.