-
Notifications
You must be signed in to change notification settings - Fork 292
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
make URL canonical instead of PDF on paper page (closes #587) #622
Conversation
This does not do the intended thing when compiling it on my machine, the result is this:
Does it work correctly on your machines? |
Yeah, it looks ugly. I was trying to separate data that's in the BibTex from other information about the paper. What about removing the border and putting the PDF line below the export formats line? |
I'd just keep it all together. The SIG info isn't in the BibTeX either. |
agree with @aryamccarthy, this is an information block and not a bibtex information block. The volume ID is also not part of the bibtex. Adding a line primarily makes people like me second-guess what the reason for the line could be :-) |
Okay, I restored it. I moved the PDF link to the bottom because it's already extraneous and it looked ugly next to the URL line (since they're nearly identical in most cases). |
Feel free to merge this if no one complains about me having moved it...:) |
@mbollmann you requested changes therefore you need to approve before it can be merged. |
…cl-org#622) * make URL canonical instead of PDF (closes acl-org#587) * class instead of hr * fixed link * removed DOI border, moved PDF to bottom
No description provided.