← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Component table view initial columns sizing

 

Hi Simon,

> On 5. Dec 2017, at 14:37, Simon Richter <Simon.Richter@xxxxxxxxxx> wrote:
> 
> Unrelated to your patch: does it make sense to perhaps share some of the
> grouping code between this and the component pin table?

First: I didn’t write that piece of code and I only had a very quick look into the table model code.
Just from GUI it looks similar (swap “pin" with “device"), so this could make sense.

However, I don’t know if it is worth the effort.
At the end, tailoring some small generic piece of code for the common model to both different tables/data might be more effort.
Maybe it would be a nice software design exercise… :)


Regards,
Bernhard



References