summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2015-09-27Add missing newlines after blocksJuuso Lapinlampi
Fixes JSCS requirePaddingNewLinesAfterBlocks warnings using Airbnb preset.
2015-09-27Fix redeclaration of "res" variable in pomf.jsJuuso Lapinlampi
Fixes ESLint warning no-redeclare.
2015-09-27Fix irregular whitespace in cheesesteak.jsJuuso Lapinlampi
Fixes ESLint warnings about no-irregular-whitespace.
2015-09-27Remove Gruntfile cssmin task banner optionJuuso Lapinlampi
The banner option was removed in grunt-contrib-cssmin v0.11.0. This removes the dead config option.
2015-09-27Remove unused expression (hits[file.name] = 0)Juuso Lapinlampi
Fixes ESLint no-unused-expressions warning.
2015-09-27Use strict node === node.parentNode comparisonJuuso Lapinlampi
Fixes ESLint no-cond-assign warning. -- Sat, 26 Sep 2015 -- 10:38:42 +bursey | while (node = node.parentNode) is invalid 10:38:54 +bursey | should be while (node == node.parentNode) 10:40:38 +bursey | weird actually 10:40:55 +bursey | Elements tend to be "true" in js 10:41:16 +bursey | but if the parentNode is undefined, then it's | false 10:41:50 +bursey | thus it doesn't loop infinitely 10:43:00 +bursey | actually, use === instead of ==
2015-09-27Fix JavaScript units[] array coding styleJuuso Lapinlampi
Reduces confusion and improves readability, despite not having yet agreed on which JavaScript style guide to use.
2015-09-27Replace copy-pasted pomf.js errors with constantsJuuso Lapinlampi
Introduces client-side UPLOAD_ERR_MAX_SIZE and UPLOAD_ERR_FAILED constants. This reduces code duplication, which means less code smell.
2015-09-27Add Cupkle to package.json contributorsJuuso Lapinlampi
Cupkle has contributed a Catalan translation, merged to master in commit e233523.
2015-09-27Simplify contributors array in package.jsonJuuso Lapinlampi
npm allows to shorten contributors into a single string, the following format: "Author Name <me@authorname.example> (http://authorname.example)" This commit changes the array to the shortened strings.
2015-09-27Remove misleading comment and simplify pomf.js case 413Juuso Lapinlampi
There appears to be no such thing as a terrible scary bug that needs a workaround in my Chromium 45.0.2454.85 browser to encode &hellip;. The unobvious self-closing <div/> is also very questionable. For what it's worth, &hellip; will encode correctly as long as you use jQuery's .html() method – which it already did anyway. OK ewhal@
2015-09-27Merge branch 'remove-moe-code-in-core' of WubTheCaptain/pomfJuuso Lapinlampi
OK ewhal@
2015-09-27Remove session_start(); from upload.phpJuuso Lapinlampi
Moe panel uses PHP sessions to keep the user logged in. We don't need it anymore, it's not required for core functionality.
2015-09-27Merge branch 'grabbing-urls-hellip' of WubTheCaptain/pomfJuuso Lapinlampi
OK ewhal@
2015-09-27Merge branch 'php-strict-comparison'Juuso Lapinlampi
OK ewhal@
2015-09-27Merge branch 'totalName' of WubTheCaptain/pomfJuuso Lapinlampi
OK ewhal@
2015-09-27Fix README.md grammar "e.g.,"Juuso Lapinlampi
A punctuation mark always precedes e.g. In American English a comma should follow e.g. The list of examples following e.g. should not be exhaustive. Reference: https://en.wiktionary.org/wiki/e.g.
2015-09-27Change README.md example domain to a.pomf.exampleJuuso Lapinlampi
.example is a reserved domain name intended to be used for documentation, defined in RFC 2606. The purpose is to reduce the likelihood of conflict and confusion. OK ewhal@
2015-09-27Add Markdown inline code blocks to README.md site/PHPJuuso Lapinlampi
Ciro Santilli's Markdown Style Guide advises to mark these as code.
2015-09-27Fix README.md grammar "an attacker"Juuso Lapinlampi
English nouns begin with an indefinite article "a" or "an". Usually "a" is used when the word that follows starts with a consontant, and "an" is used when the word that follows starts with a vowel. The word "attacker" starts with a vowel and has no different pronunciation of the first letter, so the correct indefinite article to use is "an". The correct way to pronounce it is "an attacker". This fixes an error introduced in commit db66648.
2015-09-27Fix typography on "Grabbing URLs..." with &hellip;Juuso Lapinlampi
This change may seem very subtle, but many screen readers will actually behave differently. Before this commit, a screenreader like JAWS may read "Grabbing URLs dot dot dot". After this commit, JAWS may read "Grabbing URLs" and pauses for a little longer. Reference: http://www.deque.com/blog/dont-screen-readers-read-whats-scre en-part-1-punctuation-typographic-symbols/
2015-09-27Add Pantsu.cat copyright to js/pomf.jsJuuso Lapinlampi
This compliments commit 7dca38c.
2015-09-26Rename Catalan virus scans banner as virusscans.swigJuuso Lapinlampi
bannersca.json looks for virusscans.swig, but the actual template for this has a wrong file name lacking the "-s". This fixes a bug introduced in 975e255.
2015-09-26Change virus scan banner class to alert-infoJuuso Lapinlampi
alert-info conveys a message "this needs your attention, but it's not super important." alert-error conveys a message "you screwed up, please correct the error or leave the site." This means alert-error should only be used for errors the user has made. Using it for an informational message about virus scans is incorrect.
2015-09-26Use cached totalName instead of $('.file-name', totalRow)Juuso Lapinlampi
This cleans up a little bit of smelly code, which came from a copy-paste job.
2015-09-26Remove extra two character whitespace between wordsJuuso Lapinlampi
This fixes style two character whitespace in pages (Swig) with one character whitespace. See the broken windows theory: https://en.wikipedia.org/wiki/Broken_windows_theory
2015-09-26Use <input type="hidden"> in upload_form_nojs.swigJuuso Lapinlampi
Replacing <form> action with an <input> creates an additional semantic meaning, and it's now modifiable more easily in DOM.
2015-09-26Indent upload_form_nojs.swig from spaces to tabsJuuso Lapinlampi
This creates consistency with other Swig templates.
2015-09-26Remove <br> element from upload_form_nojs.swigJuuso Lapinlampi
<br> is a presentantional element, not semantic. Use CSS instead.
2015-09-26Remove non-valid href element from #upload-btnJuuso Lapinlampi
This fixes a silly validation error that was introduced in last commit. The javascript:; is reduntant anyway.
2015-09-26Indent Gruntfile uglify task banner to two spacesJuuso Lapinlampi
Improves readability and consistency.
2015-09-26Remove reduntant <span> element in #upload-btnJuuso Lapinlampi
There seems to be no meaning to having this <span> element here with no styling applied to it. It doesn't have anything to do with drag and drop functionality either, and nothing seems to be broken after removing it.
2015-09-26Replace #upload-btn <a> element with <button>Juuso Lapinlampi
Corrects semantics and increases accessibility. The <a> element represents a hyperlink. A hyperlink is "a link from a hypertext file or document to another location or file." This is not what #upload-btn is, it's a button. Ask yourself: "Does the element change the URL?" In our case, the upload button does not so it is incorrect to use an <a> element here. The idea to use <a> elements as buttons comes from years ago when you couldn't style <button> elements in legacy browsers. Today you can, so there's no reason to break the semantics. Reference: http://blogs.ancestry.com/techroots/buttons-vs-anchors/
2015-09-26Remove <header> element from index pagesJuuso Lapinlampi
The <header> element represents a group of introductory or navigational aids. It is semantically incorrect to use it with only one element inside. The <header> element was introduced in commit 87014fa08f. That commit describes the correct usage of <header> element with multiple elements inside <header>. This bug was introduced in commit fd566d3d7f.
2015-09-26Regain sanity in nav.swig links with correct pathsJuuso Lapinlampi
Before this commit, Catalan pages had no way to return to the Catalan index page using the navigation. Instead of hardcoding pantsu.cat URLs to index, use a sane "/" path.
2015-09-26Add new Catalan translations to Gruntfile buildJuuso Lapinlampi
Previously, links to Catalan FAQ and others would be broken because the generated files would not exist. This is like a hack. Hopefully someone will soon tell me how to organize these pages to subfolders to regain my sanity.
2015-09-26Rename ca.swig as indexca.swigJuuso Lapinlampi
ca.swig is confusing and difficult to find. This commit creates consistency with other pages until someone more knowledgeable person tells me how to sort these to /pages/ca/ with Gruntfile.
2015-09-25Bump grunt-contrib-htmlmin version up to ^0.5.0Juuso Lapinlampi
This upgrades the htmlmin dependency to version 0.8.0. Unfortunately, upstream author does not maintain the CHANGELOG file.
2015-09-25Merge branch 'add-catalan-translation'Juuso Lapinlampi
Catalan translation courtesy of Cupkle. Few patches added by wub@.
2015-09-25Remove Pantsu.cat from check_fileapica.swig authorsJuuso Lapinlampi
wub@ and ewhal@ no longer have any content in this file. It's been replaced.
2015-09-25Fix HTML lang attributes for Catalan translationJuuso Lapinlampi
2015-09-25Add full, more accurate Catalan translationCupkle
2015-09-24Remove unused UploadedFile class $crc32 propertyJuuso Lapinlampi
get_crc32() was removed in commit ea6a7a0194 and never in use in the code. It seems Eric forgot to remove this property.
2015-09-24Add extra whitespace for var uploadBtn in pomf.jsJuuso Lapinlampi
This is not how it appeared in the initial commit 1792788f5f, but personally I like it and style guides allow me to adjust the whitespace for these kind of variables. Improves readability.
2015-09-24Add an extra whitespace for rowUrl in pomf.jsJuuso Lapinlampi
Improves readability by indenting these variables to same level. Restores whitespace style like in initial commit 1792788f5f.
2015-09-24Replace ambiguous MIT license mentions with ExpatJuuso Lapinlampi
MIT has used many licenses for software. Calling this project to be available under a "MIT" license is misleading.
2015-09-24Remove "The MIT License (MIT)" line from LICENSEJuuso Lapinlampi
The line is reduntant and more confusing than helpful. It belongs to README. The original Expat license text doesn't include this line.
2015-09-24HTML-ify html_error() response with <p>Eliot Whalan
2015-09-24Rename txt_error() to text_error()Eliot Whalan
2015-09-24Fix indentation to 2 spaces.Eliot Whalan
generated by cgit v1.2.3 (git 2.25.1) at 2026-10-22 15:44:16 +0200