Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

License link in footer to license #1397

Merged
merged 6 commits into from
Jul 24, 2013
Merged

License link in footer to license #1397

merged 6 commits into from
Jul 24, 2013

Conversation

tjschuck
Copy link
Member

Linking the "MIT License" footer link to the Wikipedia page is a little weird. We should probably just link to the actual license.

(Also, some indentation and whitespace fixes.)

</ul>

<footer>
&copy; 2011 <a href="http://www.getharvest.com/">Harvest</a> - Chosen is licensed under the <a href="http://en.wikipedia.org/wiki/MIT_License">MIT license</a>.
&copy; 2011 <a href="http://www.getharvest.com/">Harvest</a> &mdash; Chosen is licensed under the <a href="https://github.com/harvesthq/chosen/blob/master/LICENSE.md">MIT license</a>.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't this be 2013? Or 2011-2013?

Copy link
Member Author

Choose a reason for hiding this comment

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

I actually looked into this. It allegedly should be earliest publish date, which is 2011. But mostly, it's all stupid. I gave pause, too, and I wouldn't want to imply that this isn't maintained because of the copyright year. I'd vote for 2011-2013, depending on @pfiller's opinion.

Copy link
Contributor

Choose a reason for hiding this comment

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

My understanding is that you want the earliest publishing date as well as the years of any major revisions. Given that, I'd definitely vote for the year span. Even if it's wrong, I still prefer the "this is fresh content" signal of including the latest year.

2011-2013 it is.

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated.

@kenearley
Copy link

:shipit:

pfiller added a commit that referenced this pull request Jul 24, 2013
@pfiller pfiller merged commit d5765fd into master Jul 24, 2013
@pfiller pfiller deleted the indentation_and_footer branch July 24, 2013 21:54
pfiller added a commit that referenced this pull request Jul 26, 2013
- CSS Updates #1368, #1371
- Prevent text selection on Chosen #1374
- single_backstroke_delete defaults to true #1375
- No longer expose classes to window context #1389
- Single select value fix #1392
- Remove javascript:void and attr references #1385, #1377
- Disable Chosen on iPhone / iPod and Android mobile devices
  by default #1388
- Add Destroy Method #1396
- Option to Hide Selected & Disabled Options #1404
- Always clear result highlight #1407
- Replace characters only once #1411

- Document all the things. #1366, #1376, #1383
- Add Gemfile for compass dependency #1373
- Update copyright and license link #1397
- Add @koenpunt as a maintainer #1399
- Add anchors to documentation headers #1395
- Updated stackoverflow instructions #1403
gitlost pushed a commit to gitlost/chosen that referenced this pull request Nov 17, 2013
gitlost pushed a commit to gitlost/chosen that referenced this pull request Nov 17, 2013
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants