Mail Archives: geda-user/2012/05/28/17:19:31
On Mon, May 28, 2012 at 03:55:57PM -0500, Benjamin Bergman wrote:
> Last week I was working on a project and I wanted to get my silk screen
> looking right, but the photo-realistic options didn't match the fab house I
> wanted to use, so I patched pcb to have some options. You can find the
> patch here. https://bugs.launchpad.net/pcb/+bug/1004796
>
The code is well-written and the patches are a readable size,
which are excellent. However, I worry about the UI:
1. There are only a fixed set of colors, which limits the user, and
requires us (you) to hardcode color values. Maybe we want to accept
hex values instead?
2. What about changing the other colors? I see that you added options
for gold-plating and black silk (which IMHO are worthwhile special
cases anyway). Maybe we should accept hex values for silk and plate
colors as well?
Oh, and one other thing -- take a look at "git rebase -i", which lets
you interactively rebase and fix things like indendation without creating
a new commit.
> It is my first patch for pcb, so let me know if there is anything I should
> improve. I imagine that marking it as affecting you will also increase the
> odds of it being included into the trunk. :)
Nope, informing the mailing list (as you did) will increase your odds :).
Things often get missed on the bugtracker.
--
Andrew Poelstra
Email: asp11 at sfu.ca OR apoelstra at wpsoftware.net
Web: http://www.wpsoftware.net/andrew
"You shouldn't trust every quote you read on the Internet." -- Socrates
- Raw text -