https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17817

Patricio Marrone <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #7 from Patricio Marrone <[email protected]> ---
(In reply to Jonathan Druart from comment #6)
> (In reply to Patricio Marrone from comment #5)
> 
> Hi Patricio,
> Can you confirm that your patch fixes the following:
>  - width is wrongly calculated
>  - "Move up" raises a JS error
> ?

Hi Jonathan! Short answer: yes. 
I checked that
- The width is the same as the one from the cloned line. This was fixed by
first
  appending the cloned element to the dom and then reactivating select2, so
that
  it can calculate the width.
- Moving up should work. select2 introduces some divs which broke the present
  assumptions that all divs were lines. I added more specific selectors to
  choose what's a line and what's not. This could use some regression testing
to
  check that other kind of fields still work correctly (that is, textfields,
  textareas...). I did test that they worked to the limits of my not very
extensive
  Koha knowledge.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to