Hey John,
This looks much better. I merged your patch. Thank you for your
contribution to KiCad.
Cheers,
Wayne
On 6/25/2018 7:42 AM, John Beard wrote:
> Here is a patch for the Ki logo and a screenshot.
>
> Cheers,
>
> John
>
>
>
> On Tue, Jun 19, 2018 at 1:30 PM, Wayne Stambaugh
Here is a patch for the Ki logo and a screenshot.
Cheers,
John
On Tue, Jun 19, 2018 at 1:30 PM, Wayne Stambaugh
wrote:
> I concur. The small KiCad logo would look better.
>
> https://github.com/KiCad/kicad-source-mirror/blob/
> master/bitmaps_png/sources/kicad_logo_small.svg
>
> On
I concur. The small KiCad logo would look better.
https://github.com/KiCad/kicad-source-mirror/blob/master/bitmaps_png/sources/kicad_logo_small.svg
On 6/18/2018 6:06 AM, Nick Østergaard wrote:
> Mmm, I think I would prefer the Ki logo
>
> man. 18. jun. 2018 10.11 skrev John Beard
Mmm, I think I would prefer the Ki logo
man. 18. jun. 2018 10.11 skrev John Beard :
> In that case, here is the patch with the 48px app logo in the top left.
> Doxygen recommends 55px max height, so 48px seems a handy size.
>
> Screenshot for reference.
>
> Cheers,
>
> John
>
> On Sun, Jun 17,
On Mon, Jun 18, 2018 at 09:10:49AM +0100, John Beard wrote:
> In that case, here is the patch with the 48px app logo in the top left.
> Doxygen recommends 55px max height, so 48px seems a handy size.
>
> Screenshot for reference.
>
> Cheers,
>
> John
>
Nice,
would you mind adding a small
In that case, here is the patch with the 48px app logo in the top left.
Doxygen recommends 55px max height, so 48px seems a handy size.
Screenshot for reference.
Cheers,
John
On Sun, Jun 17, 2018 at 5:58 PM, Wayne Stambaugh
wrote:
> I'm OK with the short logo image. This would probably look
I'm OK with the short logo image. This would probably look better than
the redundant KiCad.
On 06/17/2018 08:53 AM, John Beard wrote:
> Sadly I don't think there is a way to set the logo alt text in doxygen
> (so it would still read "KiCad" when the image is not displayed, for
> screen readers,
Sadly I don't think there is a way to set the logo alt text in doxygen (so
it would still read "KiCad" when the image is not displayed, for screen
readers, when copy-pasted and it when search engines index it).
Alternatively, use the square icon rather than the text logo and keep the
title as it
I agree. The double KiCad does look a bit odd but I do like the KiCad
logo embedded in the docs.
On 06/13/2018 06:00 PM, Nick Østergaard wrote:
Maybe we need to remove the word KiCad from the title, as it
essentially reads "[KiCad] KiCad Pcbnew Python Scripting"
2018-06-13 21:12
Maybe we need to remove the word KiCad from the title, as it essentially
reads "[KiCad] KiCad Pcbnew Python Scripting"
2018-06-13 21:12 GMT+02:00 John Beard :
> Sorry, I had a line commented in a local commit to avoid a costly re-build
> of the docstrings when I didn't need them to be (the
Sorry, I had a line commented in a local commit to avoid a costly re-build
of the docstrings when I didn't need them to be (the doc dependencies
aren't managed by CMake, it just always runs the targets, which is fine for
CI).
The attached should be better (difference is a lack of a "# " before
John,
I am getting the following error when I try to apply your patch:
$ git am --ignore-whitespace <
/c/Users/wstambaugh/Documents/Archive/temp/kicad/patches/0001-Documentation-add-logo-to-Doxygen-pages.patch
Applying: Documentation: add logo to Doxygen pages
error: patch failed:
Hi,
This patch adds a small KiCad logo to the three doxygen documentation
targets (C++, dev docs and Python docs) for a little bit more flair.
The image itself is put in the /Documentation dir, but it's not really
important to the process where it goes.
Cheers,
John
From
13 matches
Mail list logo
