← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH 1/7] Clear comp highlighting before highlight new one.

 

Hi Seth. Thank you for your attention.

1) I admit I made a bad choice and now I'm working on new patch with which
either Esc closes a cell editor if it's active or Esc closes dialog if a
cell editor doesn't active.

2) This patch is fixed and sent to accoring thread ([PATCH 7/7] Grid
tricks: paste to selection.).

4) I cannot reproduce it but it may be caused by 2).

3, 5) These problems present in master. Proposed patches don't have a
putpose to fix *all* problems. There is still work to do.

ср, 16 янв. 2019 г. в 21:32, Seth Hillbrand <seth@xxxxxxxxxxxxx>:

> Am 2019-01-16 09:56, schrieb Wayne Stambaugh:
> > Konstantin,
> >
> > What platform did you use to test your changes?  If it was linux, are
> > you using gtk2 or gtk3?  Would some of our other devs please test this
> > patch set.  wxGrid tends to have it's own set of platform dependent
> > quirks so I would like to avoid breaking anything before merging these
> > changes.
>
>
>
> This part works for me on Debian Buster/gtk3.
>
> I have a couple comments on the patch set.
>
> 1) Shift-Esc is not intuitive and we shouldn't use it.  Escape to exit
> the edited cell and restore the value is very nice but if not actively
> editing a cell, escape should close the dialog.  The prompt for unsaved
> changes should be sufficient to prevent accidental closure.
>
> 2) Line 456 of grid_tricks.cpp uses end_col for the loop end condition.
> But it is only initialized if there is a selection.  Otherwise, it is
> undefined.
>
> 3) Pasting into a collapsed row should (ideally) expand the row.
> Currently, it writes into the collapsed row, which fills all sub rows
>
> 4) Pasting into the Configure Paths dialog goes into an infinite loop.
>
> 5) Pasting the wrong column into the Pin Table is a segfault (Copy from
> one column, paste into another
>
> -Seth
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>

Follow ups

References