-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/ -----------------------------------------------------------
Review request for GGD Tech Group.
Summary
-------
This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.comics.org/r/1027/ > -----------------------------------------------------------
> Review request for GGD Tech Group.
> Summary
> -------
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
Don't worry, take your time. I assumed that you were busy.
By the way, here's some more info on the change for when look at the
code:
The interesting JS code is in media/js/oi/revision_form_utils.js.
I factored out the jQuery loading in templates/oi/bits/jquery.html
I added a new block 'footer' in templates/gcd/base.html so that any
JS loading can go in the end of the page - this seems to be better,
since it avoids blocking the rendering of the page if for some reason
the JS is slow to load, as I read in many articles on how to deploy
jQuery.
I'm passing the synopsis maximum length as a variable - it was hardcoded
in a couple of places. This necessitated adding the settings to the
request context in some cases.
The reason I updated all JS libraries at the same time is that since I
was going to test various browsers for compatibility problems anyway, it
made sense to do those updates at the same time, to consolidate testing.
> too busy till next week to look at the code (it might be simple, but I
> just won't look at code reviews).
> Jochen
> Am 30.08.2012 03:01, schrieb Alexandros Diamantidis:
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > http://reviews.comics.org/r/1027/ > > -----------------------------------------------------------
> > Review request for GGD Tech Group.
> > Summary
> > -------
> > This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> > Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> > Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/#review2101 -----------------------------------------------------------
I don't know if we really want to support IE < 9 by special fixes. It is less than 15% and dropping.
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.comics.org/r/1027/ > -----------------------------------------------------------
> (Updated 2012-08-30 00:01:51)
> Review request for GGD Tech Group.
> Summary
> -------
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
> On 2012-09-10 18:48:24, Jochen G. wrote:
> > I don't know if we really want to support IE < 9 by special fixes. It is less than 15% and dropping.
Well, it doesn't cost us much. The main incompatibility here concerns the events that fire when someone types in a textarea or input field. With newer browsers you can just bind the "input" event, while in IE < 9 and I think somewhere else you need to bind other events as well. The html5fixes plugin canonicalizes this by simulating the "input" event where it doesn't exist, so there's no need for special fixes in our code.
Of course if we encounter any real difficulties, we can always change our minds ;)
// Disable form submission with Enter for barcode field
$('#id_barcode').keypress(function (e) {
if (e.which == 13) {
return false;
}
});
This way, we don't need to put any JS code in our forms.
If we want to do this for other fields too, we can just add their id's (or classes) in the $('#id_barcode') selector. And if the situation becomes more complex, we can even write a small jQuery plugin for this.
> > if we call it footer, it should be after the body, or ?
Well, maybe "footer" isn't the best name - mainly it's the place where we'll put the scripts if there's no reason to put them elsewhere. I just saw some people calling this place the "footer" in the articles I read, but we could also call it "bottom", "page_end" or something similar.
Now, in our templates I think the proper place to load the JavaScript is right next to the CSS, even if (now that I've added the "footer" block) they'll appear in a different part in the final HTML output. I think this grouping makes it a bit cleaner, and the placement doesn't really matter to Django.
- Alexandros
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/#review2101 -----------------------------------------------------------
On 2012-08-30 00:01:51, Alexandros Diamantidis wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.comics.org/r/1027/ > -----------------------------------------------------------
> (Updated 2012-08-30 00:01:51)
> Review request for GGD Tech Group.
> Summary
> -------
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
> On 2012-09-10 18:48:24, Jochen G. wrote:
> > I don't know if we really want to support IE < 9 by special fixes. It is less than 15% and dropping.
> Alexandros Diamantidis wrote:
> Well, it doesn't cost us much. The main incompatibility here concerns the events that fire when someone types in a textarea or input field. With newer browsers you can just bind the "input" event, while in IE < 9 and I think somewhere else you need to bind other events as well. The html5fixes plugin canonicalizes this by simulating the "input" event where it doesn't exist, so there's no need for special fixes in our code.
> Of course if we encounter any real difficulties, we can always change our minds ;)
It costs us an additional file to maintain/have with little gain. These changes are nice, but uncritical, so we don't loose much by not having these for the 15% (and dropping) affected.
> > if we call it footer, it should be after the body, or ?
> Alexandros Diamantidis wrote:
> Well, maybe "footer" isn't the best name - mainly it's the place where we'll put the scripts if there's no reason to put them elsewhere. I just saw some people calling this place the "footer" in the articles I read, but we could also call it "bottom", "page_end" or something similar.
> Now, in our templates I think the proper place to load the JavaScript is right next to the CSS, even if (now that I've added the "footer" block) they'll appear in a different part in the final HTML output. I think this grouping makes it a bit cleaner, and the placement doesn't really matter to Django.
It is recommended to load javascript as the last part of a webpage, or ? So it can be called footer, as it should be the last part of the page.
JS doesn't really have much to do with CSS. The order of things in our templates should reflect the order in which is loaded and the order in the base.html, otherwise it can get confusing.
- Jochen
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/#review2101 -----------------------------------------------------------
On 2012-08-30 00:01:51, Alexandros Diamantidis wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.comics.org/r/1027/ > -----------------------------------------------------------
> (Updated 2012-08-30 00:01:51)
> Review request for GGD Tech Group.
> Summary
> -------
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/#review2106 -----------------------------------------------------------
I still don't really like the additional file for IE < 9, we don't necessarily need to add this functionality for the older browsers which disappear anyway in time. So I would prefer to remove it.
If you can answer my other question ?
Otherwise I would like to have the footer actually as the last part of the page, after body. If I am not mistaken this is where javascript should be put, so that it doesn't hinder the loading of the rest.
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.comics.org/r/1027/ > -----------------------------------------------------------
> (Updated Aug. 30, 2012, 12:01 a.m.)
> Review request for GGD Tech Group.
> Description
> -------
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
> I still don't really like the additional file for IE < 9, we don't necessarily need to add this functionality for the older browsers which disappear anyway in time. So I would prefer to remove it.
> If you can answer my other question ?
> Otherwise I would like to have the footer actually as the last part of the page, after body. If I am not mistaken this is where javascript should be put, so that it doesn't hinder the loading of the rest.
> // Disable form submission with Enter for barcode field
> 95
> $('#id_barcode').keypress(function (e) {
> 96
> if (e.which == 13) {
> 97
> return false;
> 98
> }
> 99
> });
> Is the separate handling for IE is not necessary anymore ?
> - Jochen
> On August 30th, 2012, 12:01 a.m., Alexandros Diamantidis wrote:
> Review request for GGD Tech Group.
> By Alexandros Diamantidis.
> *Updated Aug. 30, 2012, 12:01 a.m.*
> Description
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Testing
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
> --
> GCD-Tech mailing list - gcd-tech@googlegroups.com
> To unsubscribe send email to gcd-tech+unsubscribe@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/gcd-tech
> On Oct. 2, 2012, 8:59 p.m., Jochen G. wrote:
> > I still don't really like the additional file for IE < 9, we don't necessarily need to add this functionality for the older browsers which disappear anyway in time. So I would prefer to remove it.
> > If you can answer my other question ?
> > Otherwise I would like to have the footer actually as the last part of the page, after body. If I am not mistaken this is where javascript should be put, so that it doesn't hinder the loading of the rest.
Sorry for dragging my feet on this... I intend rework it a bit and submit an updated request. I'll move the JS loading as you indicated and use a different jQuery plugin for the input events - see: http://whattheheadsaid.com/projects/input-special-event - if you read the comments there, you'll see that even IE9 has some problems with input events and this tries to work around them.
> > Is the separate handling for IE is not necessary anymore ?
Yes, the nice thing about jQuery's keypress events is that they normalize the property that holds they key - so no separate handling needed.
- Alexandros
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/#review2106 -----------------------------------------------------------
On Aug. 30, 2012, 12:01 a.m., Alexandros Diamantidis wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.comics.org/r/1027/ > -----------------------------------------------------------
> (Updated Aug. 30, 2012, 12:01 a.m.)
> Review request for GCD Tech Group.
> Description
> -------
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/ -----------------------------------------------------------
(Updated Oct. 5, 2012, 12:17 p.m.)
Review request for GCD Tech Group.
Changes
-------
Added more languages to the month-matching regular expressions. They should now cover most publication dates we have in the db that can be interpreted unambiguously.
Moved the JS loading to the end of the templates.
Replaced html5fixes with input-special-event - it seems better-maintained and ensures compatibility across all browsers. It's true that as browsers get updated and native compatibility improves that won't be needed, but I still think that being non-intrusive and low-maintenance for us, making sure everything works even for older browsers has value.
Description
-------
This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/#review2109 -----------------------------------------------------------
Hmm, you forgot the new files, or ?
And btw, the updates of the existing jquery thingies don't need to be in the review, it's not like we are reviewing them. The one new external one is of some interest, since we add it, but a link to a webpage or so would also be fine.
- Jochen G.
On Oct. 5, 2012, 12:17 p.m., Alexandros Diamantidis wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.comics.org/r/1027/ > -----------------------------------------------------------
> (Updated Oct. 5, 2012, 12:17 p.m.)
> Review request for GCD Tech Group.
> Description
> -------
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
> On Oct. 5, 2012, 11:57 p.m., Jochen G. wrote:
> > Hmm, you forgot the new files, or ?
> > And btw, the updates of the existing jquery thingies don't need to be in the review, it's not like we are reviewing them. The one new external one is of some interest, since we add it, but a link to a webpage or so would also be fine.
I did, sorry! Got confused moving the chanhes to git from SVN.
I thought I'd include all JS libraries as a way to allow anyone testing this change to install them easily (by just applying the diff from this review) but you're right that it makes the review request much more confusing. So, I did as asked and just mention them in the description.
- Alexandros
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/#review2109 -----------------------------------------------------------
On Oct. 5, 2012, 12:17 p.m., Alexandros Diamantidis wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.comics.org/r/1027/ > -----------------------------------------------------------
> (Updated Oct. 5, 2012, 12:17 p.m.)
> Review request for GCD Tech Group.
> Description
> -------
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/ -----------------------------------------------------------
(Updated Oct. 7, 2012, 2:37 a.m.)
Review request for GCD Tech Group.
Changes
-------
Hopefully everything is correct now. Will also add latest jQuery (1.8.2), Jcrop (0.9.10) and input-special-event 1.1 (from: http://whattheheadsaid.com/projects/input-special-event), which are not included in this review..
Description
-------
This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.comics.org/r/1027/#review2111 -----------------------------------------------------------
Ship it!
This looks good now.
Please submit to the git, but don't deploy, the change from svn to git still needs to be done on production, but we have local changes due to adds, which won't be committed for now...
Thanks.
- Jochen G.
On Oct. 7, 2012, 2:37 a.m., Alexandros Diamantidis wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.comics.org/r/1027/ > -----------------------------------------------------------
> (Updated Oct. 7, 2012, 2:37 a.m.)
> Review request for GCD Tech Group.
> Description
> -------
> This change began as an addition of two small UI improvements using jQuery: A running length indication while entering the synopsis and a function to automaticaly detect the keydate from the pubdate field (per bug #684).
> Because the easiest way to process input while typing is the "input" JS event which isn't supported in IE < 9, I imported the HTML5fixes jQuery plugin which simulates it when needed. I also took the liberty of updating our other JS libraries to the latest versions (jQuery v1.7.2, Jcrop v0.9.10) - not actually needed, but it doesn't hurt.
> Tested the new functions and the gatefold cover selection (Jcrop) on Firefox, Opera, Chromium and IE 6 and 10-preview. The only problem on IE6 is that if you paste something in the synopsis using only the mouse (not Ctrl-V for example) the event doesn't fire and the synopsis length isn't updated. Just clicking on the textarea or pressing a key updates it.
