Re: [Spacewalk-devel] [PATCH] Date/Time picker

2014-02-10 Thread Silvio Moioli
On 02/06/2014 11:12 AM, Duncan Mac-Vicar P. wrote:
 See attached patches. [...]

On a slightly unrelated note, I have just noticed the current date
picker allows incorrect dates to be picked, such as February 31. In the
SSM errata scheduling page, among other things, such values cause a NPE.

Regards,
-- 
Silvio Moioli
SUSE LINUX Products GmbH
Maxfeldstraße 5, 90409 Nürnberg Germany

___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel


Re: [Spacewalk-devel] UI: Errata pages ASAP removal missing patches

2014-02-10 Thread Matej Kollar
Pushed to master as 1ae33e052ef7b449ed34b8bea9411bc45631c0e6.
Two little things were fixed in following two commits:

  * 31d358af3f9707771107ed6887989d30e50f
Consistent closing of void tags
(as you said that you completely rewritten the
page so I thought it would be nice co have it
consistent).
  * 6303cdfbff967d124c8113b1ecef534ca71c610c
While checking how the page looks you probably
missed doubled colons. 

Thanks for contribution :-).
--
Matej Kollar
Satellite Engineering, Red Hat

- Original Message -
 From: Bo Maryniuk b...@suse.de
 To: Bo Maryniuk b...@suse.de, spacewalk-devel@redhat.com
 Sent: Monday, February 10, 2014 3:40:58 PM
 Subject: Re: [Spacewalk-devel] UI: Errata pages ASAP removal missing patches
 
 Hi,
 errata fix (again), must be applicable this time.
 
 --
 Bo Maryniuk
 
 SUSE LINUX Products GmbH, HRB 16746 (AG Nürnberg)
 GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer
 
 No matter where you go in the world, there never
 seems to be a shortage of idiots.
 
 ___
 Spacewalk-devel mailing list
 Spacewalk-devel@redhat.com
 https://www.redhat.com/mailman/listinfo/spacewalk-devel

___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Re: [Spacewalk-devel] UI: Errata pages ASAP removal missing patches

2014-02-10 Thread Bo Maryniuk
On Mon, Feb 10, 2014 at 07:49:48AM -0500, Matej Kollar wrote:
 Nevertheless I have a note for mentioned #0009: If
 you really insist on reformatting .jsp-s, please
   * do it in separate step to make review simpler,
   * do not introduce trailing white-spaces,
   * make consistent use of space before slash in void tags,
   * make indentation of pairs of opening and closing tags match.

This sounds like need of a pretty precise checkstyle. Well, fixed.

-- 
Bo Maryniuk

SUSE LINUX Products GmbH, HRB 16746 (AG Nürnberg)
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer

He can compress the most words into the smallest idea of any man I know.


datepicker-ui-unification-errata-pages.tar.bz2
Description: application/bzip-compressed-tar
___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel