Inconsistency in Pcbnew General Settings dialog tooltips

Bug #1591428 reported by Evan Shultz
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
Low
Chris Pavlina

Bug Description

When viewing the tooltips in the General Settings dialog in Pcbnew, there are a number of inconsistencies.

Here are a few:
1. Most tooltips end with a period, but many do not (Magnetic Pads and Magnetic Tracks, for example).
2. The tooltip for Magnetic Pads does not start with a capital letter.
3. There is a line break ("\n") in the tooltip for Coordinates, but it appears the tooltip automatically wraps to a new line so this results in a weird break in the text (see attachment).
4. In the Coordinates section, the first item is "Cartesian coordinates"; the last letter is cut off (see attachment again).

Checking GitHub, the he affected file is kicad-source-mirror/pcbnew/dialogs/dialog_general_options_BoardEditor_base.cpp.

To clarify the version information below, I'm using Windows 10 with 3200x1800 resolution and 150% scaling.

Application: kicad
Version: 4.0.2-stable release build
wxWidgets: Version 3.0.2 (debug,wchar_t,compiler with C++ ABI 1009,GCC 5.2.0,wx containers,compatible with 2.8)
Platform: Windows 8 (build 9200), 64-bit edition, 64 bit, Little endian, wxMSW
Boost version: 1.57.0
         USE_WX_GRAPHICS_CONTEXT=OFF
         USE_WX_OVERLAY=OFF
         KICAD_SCRIPTING=ON
         KICAD_SCRIPTING_MODULES=ON
         KICAD_SCRIPTING_WXPYTHON=ON
         USE_FP_LIB_TABLE=HARD_CODED_ON
         BUILD_GITHUB_PLUGIN=ON

Revision history for this message
Evan Shultz (evan-shultz) wrote :
Revision history for this message
Evan Shultz (evan-shultz) wrote :

Another example screenshot.

Revision history for this message
Chris Pavlina (pavlina-chris) wrote :

All of that's going to be ripped apart and redone soon, so this should all get fixed :)

Changed in kicad:
assignee: nobody → Chris Pavlina (pavlina-chris)
status: New → Triaged
importance: Undecided → Low
Revision history for this message
Evan Shultz (evan-shultz) wrote :

Great! Thanks Chris.

In a similar vein, in Eeschema, the tooltip for the "Add image" command is "Add bitmap image" even though many formats besides BMP are supported. The menu option for this command is Place > Image.

Revision history for this message
xzcvczx (xzcvczx) wrote :

but a bitmap is not only bmp. And vectors are not supported afaik

Revision history for this message
Chris Pavlina (pavlina-chris) wrote : Re: [Bug 1591428] Re: Inconsistency in Pcbnew General Settings dialog tooltips

Agreed, "bitmap" is just another term for "raster" and that's the only
image type supported. Nothing wrong there.
On Jun 11, 2016 06:20, "Simon Wells" <email address hidden> wrote:

> but a bitmap is not only bmp. And vectors are not supported afaik
>
> --
> You received this bug notification because you are a bug assignee.
> https://bugs.launchpad.net/bugs/1591428
>
> Title:
> Inconsistency in Pcbnew General Settings dialog tooltips
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/kicad/+bug/1591428/+subscriptions
>

Revision history for this message
Novak Tamas (novak-7) wrote :

I only call "bitmap" an uncompressed bitmap: I mean JPG and some other raster but compressed formats are not "bitmaps" for me.
"Add image" would be more exact..and simpler.

Revision history for this message
jean-pierre charras (jp-charras) wrote :

I fully disagree: a bitmap image is a matrix of pixels.
The opposite is a vectorized image.

Compressed formats are just a way to reduce the size of files: they can be applied to bitmaps and vectorized images.

Kicad use both images.

Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

Fixed in revision af88781262ef8cbc45506075f3a9080225e5c05f
https://git.launchpad.net/kicad/patch/?id=af88781262ef8cbc45506075f3a9080225e5c05f

Changed in kicad:
status: Triaged → Fix Committed
Revision history for this message
Chris Pavlina (pavlina-chris) wrote :

I didn't do anything about the label being cut off as I think that's actually a wxWidgets bug.

Changed in kicad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.