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

DRY tags_max calculation in _layouts/tags.html #1696

Merged
merged 1 commit into from
Jun 4, 2018
Merged

DRY tags_max calculation in _layouts/tags.html #1696

merged 1 commit into from
Jun 4, 2018

Conversation

e28eta
Copy link
Contributor

@e28eta e28eta commented Jun 3, 2018

It was being done twice, and it doesn't seem to need to be.

It was being done twice, and it doesn't seem to need to be.
@e28eta
Copy link
Contributor Author

e28eta commented Jun 3, 2018

Again, thanks for all the hard work you do on this theme! As I was working on #1697, I saw this potential opportunity to clean it up.

I wasn't sure if you'd want both of these changes, so for simplicity I split it up.

@mmistakes
Copy link
Owner

Thanks for this. Looks good!

@mmistakes mmistakes merged commit e366a9a into mmistakes:master Jun 4, 2018
luispuerto referenced this pull request in luispuerto/luispuerto-net Jun 27, 2018
It was being done twice, and it doesn't seem to need to be.
kkunapuli pushed a commit to kkunapuli/kkunapuli.github.io that referenced this pull request May 30, 2019
It was being done twice, and it doesn't seem to need to be.
sumeetmondal pushed a commit to sumeetmondal/sumeetmondal.github.io that referenced this pull request Sep 10, 2019
It was being done twice, and it doesn't seem to need to be.
@e28eta e28eta deleted the dry-tags-max branch May 23, 2020 06:40
jchwenger pushed a commit to jchwenger/jchwenger.github.io that referenced this pull request May 5, 2023
It was being done twice, and it doesn't seem to need to be.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants