kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #29506
Re: [PATCH] Bugfix for component table
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Tue, 23 May 2017 09:10:02 -0400
-
In-reply-to:
<CA+Mgg7MGA1Goty4Mw3jgsRjfE=VCzNdyxatnOcgEZHn-XWxZnw@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1
On 5/23/2017 4:17 AM, Fabrizio Tappero wrote:
> cheers Oliver,
> I do not know what a .dcm file is.
>
> I am not sure it is a good idea to have a in-built field called
> Description in the table viewer. But I can easily see how that is
> useful. I guess you would like to use this table to add info. This makes
> it non a table viewer. The current icon label says "Component table
> view" which I find ambiguous. Maybe Wayne can jump in with a proper name
> for it.
The component information strings are not fields. They are part of the
component definition and are not user definable. There are four
component information definitions: name, description, key words (tags),
and documentation file name. You could create fields with the same
names so I can understand how this could be confusing.
Toolbar button tooltips should be a verb followed by a short
description. In this case "Edit component properties" would work.
>
> I am having the feeling the development of this table did not come from
> a clean plan of having an XYZ tool. Or maybe is just me. Please dont get
> me wrong, I love this table (and the soon to come new icon) but it might
> be a good idea to drop the word viewer for it if we want the (great)
> ability to edit stuff.
>
> I hope this helps.
>
> Fabrizio
>
>
>
>
>
>
>
>
>
> On Tue, May 23, 2017 at 10:00 AM, Oliver Walters
> <oliver.henry.walters@xxxxxxxxx <mailto:oliver.henry.walters@xxxxxxxxx>>
> wrote:
>
> Fabrizio,
>
> 1. You have added a custom field called "Description".
> 2. The table viewer has an "inbuild" field called "Description". It
> populates this field with the component description which is found
> in the .dcm files
>
> Here's what I see.
>
> Inline image 1
>
> Here I have added the custom "description" field with dummy data to
> only one component.
>
> The first "Description" column is filled with the symbol
> descriptions from the .dcm files in the library.
>
> If you are missing the .dcm files then your first column will be empty.
>
>
> On Tue, May 23, 2017 at 5:57 PM, Fabrizio Tappero
> <fabrizio.tappero@xxxxxxxxx <mailto:fabrizio.tappero@xxxxxxxxx>> wrote:
>
> Hi Oliver,
> not sure I understand the question. This is what my schematic
> components fields look like:
>
> Inline image 1
>
>
> cheers
> Fabrizio
>
>
> On Tue, May 23, 2017 at 9:53 AM, Andrey Kuznetsov
> <kandrey89@xxxxxxxxx <mailto:kandrey89@xxxxxxxxx>> wrote:
>
> Previous bug even works without modifying a field, all I
> have to do is enter a field edit mode and click somewhere
> else, and that field's value will be copied to where ever I
> clicked, this will cause a lot of grief when users decide to
> cancel editing and click elsewhere to discard changes, or so
> they think that's what should have happened.
>
>
> A few more things:
> 1. When I click Apply changes, the RED markings/modified
> fields should disappear when the changes have been
> successfully applied
> 2. Undo/Redo should work inside Component Table to undo each
> user action, instead of having the user to discard all
> changes and start over
> 3. There's something weird going on when you try to edit a
> field, it seems that have to click 4 times to enter edit
> mode, 2 times is what I expect. Also perhaps some weird
> delay that prevents me from entering edit mode.
>
> On Tue, May 23, 2017 at 12:39 AM, Andrey Kuznetsov
> <kandrey89@xxxxxxxxx <mailto:kandrey89@xxxxxxxxx>> wrote:
>
> Oliver, found another unexpected behaviour.
> When I select a field to edit, for example to replace a
> "Part Number" field from AAA to BBB, when I type in BBB
> but instead of hitting enter, I click on the next field
> below it which belongs to another part, then BBB
> modifies that part's field instead of where the editing
> took place.
>
> On Tue, May 23, 2017 at 12:28 AM, Oliver Walters
> <oliver.henry.walters@xxxxxxxxx
> <mailto:oliver.henry.walters@xxxxxxxxx>> wrote:
>
> The "Description" field should be showing the
> description of each symbol as it appears in the .dcm
> file. It was working for me - do your symbols not
> have those descriptions?
>
> On Tue, May 23, 2017 at 5:23 PM, Fabrizio Tappero
> <fabrizio.tappero@xxxxxxxxx
> <mailto:fabrizio.tappero@xxxxxxxxx>> wrote:
>
> Hi Guys,
> great ! you guys almost fixed it. It looks like
> the column "Description" appears twice and one
> of the two is completely empty.
>
> cheers
> Fabrizio
>
>
>
> Inline image 1
>
> On Mon, May 22, 2017 at 10:18 PM, Wayne
> Stambaugh <stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>> wrote:
>
> Oliver,
>
> I committed your patch. Thank you.
>
> Wayne
>
> On 5/22/2017 10:08 AM, Oliver Walters wrote:
> > Bug noted here -
> https://lists.launchpad.net/kicad-developers/msg29485.html
> <https://lists.launchpad.net/kicad-developers/msg29485.html>
> >
> > Patch attached to this email fixes glitch
> when user adds custom field
> > with same name as a default field.
> >
> > Users can now do this to their heart's
> content.
> >
> >
> >
> _______________________________________________
> > Mailing list:
> https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > Post to :
> kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > Unsubscribe :
> https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > More help :
> https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> >
>
> _______________________________________________
> Mailing list:
> https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> Post to :
> kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> Unsubscribe :
> https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> More help :
> https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
>
>
>
> _______________________________________________
> Mailing list:
> https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> Post to :
> kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> Unsubscribe :
> https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> More help :
> https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
>
>
>
> _______________________________________________
> Mailing list:
> https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> Unsubscribe :
> https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> More help : https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
>
>
>
>
> --
> Remember The Past, Live The Present, Change The Future
> Those who look only to the past or the present are
> certain to miss the future [JFK]
>
> kandrey89@xxxxxxxxx <mailto:kandrey89@xxxxxxxxx>
> Live Long and Prosper,
> Andrey
>
>
>
>
> --
> Remember The Past, Live The Present, Change The Future
> Those who look only to the past or the present are certain
> to miss the future [JFK]
>
> kandrey89@xxxxxxxxx <mailto:kandrey89@xxxxxxxxx>
> Live Long and Prosper,
> Andrey
>
>
>
>
>
>
> _______________________________________________
> 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