kicad-developers team mailing list archive
Mailing list archive
Re: [Patch] cvpcb Usability Improvements
Wayne Stambaugh <stambaughw@xxxxxxxxx>
Fri, 14 Jun 2019 15:18:04 -0400
addr=stambaughw@xxxxxxxxx; prefer-encrypt=mutual; keydata= mQGiBEM0hxQRBAC2fNh3YOVLu1d5GZ0SbrTNldGiGnCJPLqzEnqFX9v6jmf33TMt6EmSLkl6 Wtfkoj0nVwKxcYmJkA8DX0QAokBkwNIzhSsBzQvthBLIk/5LnPVVKrEXOcL4mUyH1doKlkaE slgJozNa6Av+oavcvD02o1zJOloBbaHlNlyRt7fKswCgtIFlVjWggVH/15KfWk+Qo5JVPbME AIUBAQyL2OAx0n60AWec2WHnO9buHuG0ibtICgUMkE+2MRmYyKwYRdyVwGoIUemFuOyHp0AJ InX4T+vy2E7vkwODqjtMLfIoRkokW74Fi4nrvjlhOAw/vdq/twLbAmR9MOfPTpR4y7kQy1O2 /n+RkkRvh26vTzfbQmrH7cBJhk6aA/9Uwvu3E4zNJgHVZeS0HyWtmR1eOPPRbnkPgJTToX5O KMKzTJI/FX6kT7cFoCamitHrW3BJP4Dx+cMMsa47EGxqVTdbVJ4LjogsXTXxb+0Fn1u4zBdx x3Cer6O7+hqWy7zvpzeC6nSREjqDKa5CgHtv/GLm5uFPOmsjAsnHj2tlBrQmV2F5bmUgU3Rh bWJhdWdoIDxzdGFtYmF1Z2h3QGdtYWlsLmNvbT6IeAQTEQIAOBYhBOffs6CbblRzBkv33BtR cWlZ+CReBQJbFBS2AhsDBQsJCAcCBhUKCQgLAgQWAgMBAh4BAheAAAoJEBtRcWlZ+CReMI8A nRbrLkzp7+c2f0vX7sfg4ICX8LAKAJ9uClo4uJajmZa5zZrL2nKdZlUwIrkCDQRDNIcxEAgA gCru+3/aOC6RCjpvYC72wY+d5SmHphC6yeiV2/mOumyt5MLo/Ps2GznZr11JspqFk5K/Zpvp MMLqqjDZ39+50a2iKRQFJ6NlK+hJWMmj6eJygQrCwYo3Gjc6CqfrqUv+8VSnf/i5sIZmtOVA 4ZjML18MuBvMSsNdVLFJd5HNnYb1iOECpvqdPVh/21LLCEw7MUUGGnHBhCrmk2aJe5hFmcSN g4ldBcXrgMQBwf7aMVoobXBMFDb/IENByXn0llB7Gr2IFMRmNS9/p8s/II1Yl2bTqyX4FSz8 cfn7C9KEz7faZ7wzAcpwHFC/zs3JoAjJ0IEKdNUpIwAlKMzT3CzctwADBQf/cxpG28MKyrqk nNmq/8LQLy+x6FSYXBLjxQz9BiBNYeesDZQ6J5UbL1mjpJzMa5tLZypPYo4bbGyR22hrbyDF K7m6AcVaMIJKl98g4ukMutFfAJyRDaREH5Zl/X1P4u1Z/yaAIy9mKaNbaK1/5djNJ5wCTFen TUgAp9xdc30kGkFDdLJFp5uxDY4P0vaZiZdjUCvDM3Zjv5IzpNOfxVqTUBQNUP/BnnKhkk0p DTD6s3X8S+D0rOtEBQ8K0cwERI/E8EFa8nj0TNw4e2MYGR8wg+SxqJ7z5f0zPY0bO6G9DDFB wYCqzzPWGqdAh9vA5971TAbPERtdFybhkurozp2SfYhJBBgRAgAJBQJDNIcxAhsMAAoJEBtR cWlZ+CResHUAniULLCWiT26ieRTl7N2vS6vBo/DuAJ4m7Ss/gyiW6ybTn1ctDXAUgm2QVQ==
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0
On 5/31/19 6:13 PM, Ian McInerney wrote:
> Attached is a patchset that includes some upgrades to the usability of
> cvpcb, namely the following:
> * Include a line below the footprint description giving the URI of the
> source library. This allows people to see where the library is from. It
> is located in a 3rd line below the footprint description, since I
> figured this placement is the least obtrusive. It is also only displayed
> for the selection from the footprint list (not the component footprint)
> since the description field is not displayed for the component.
> (requested in https://bugs.launchpad.net/kicad/+bug/1782805)
I'm OK with this patch but I'm not sure it completely works the way
users will expect depending on the footprint filtering. For example, if
the library filter is on and I select a symbol that already has an
assigned footprint from a different library, no library URI shows up in
the status pane. I would expect that the shown library would always be
the library of the currently selected symbol. Maybe I'm not
understanding the intent of this feature.
> * Implement the ability for the escape key to close the window, since it
> has a cancel button on it this seems like a useful feature. This will
> prompt the user to save the changes if there are any, and will only
> operate when the assignment window has focus, so if escape is pressed in
> the footprint viewer window it will not close the main window.
> (requested in https://bugs.launchpad.net/kicad/+bug/1830483)
I'm fine with this patch although it does not apply cleanly to the
master branch. I'll merge it into 5.1 if no one objects.
> * Implement the ability to delete individual footprint associations
> using the delete key or a right-click menu option. (requested
> in https://bugs.launchpad.net/kicad/+bug/1818883)
> * Implement a cut/copy/paste system for the footprint associations that
> operates on a 1->1 or 1->many arrangement. When more than 1 component is
> selected for the copy/cut command, only the 1st selected one is used. It
> can paste this to any number of selected components. This is also
> available through the right-click menu. (requested
> in https://bugs.launchpad.net/kicad/+bug/1794883).
I like this patch as well. As with patch 2, it does not cleanly apply
to the master branch, I suspect users will want undo/redo capability
with this because I found myself hitting the undo short cut a couple of
times while testing. As is, it's a worthwhile change. I'll merge it
into 5.1 if no one objects.
> These changes work for both master and 5.1, and I think would be useful
> to include in 5.1.3 (one of those reports requesting the features is
> actually already targeted to 5.1.3).
> 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