kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #39068
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