Opened 3 years ago

Closed 18 months ago

#14042 closed enhancement (fixed)

wxGridCellTextEditor unusable in wxOSX/Cocoa

Reported by: fsenore Owned by:
Priority: normal Milestone: 3.0.0
Component: wxGrid Version: stable-latest
Keywords: wxGrid editor Cocoa Cc:
Blocked By: Blocking:
Patch: no

Description

wxGrid cell text editors look rather bad under OS X. Just run the grid sample and type in any cell: the editor is too large, much larger than the grid cell. Moreover it has that blue line that surrounds the active control that makes it look strange and even larger. This is not what a spreadsheet user would expect. It would be nice to avoid at least the blue highlight.

Moreover in the Cocoa port the editor shows something like a grey scrollbar at the right of the control, even if it is not multiline. It would be nice to avoid it too.

Change History (8)

comment:1 Changed 3 years ago by fsenore

  • Version set to 2.9-svn

comment:2 Changed 2 years ago by SC

(In [71783]) trying to improve appearance of gridcelltexteditor under OSX, see #14042

comment:3 Changed 2 years ago by fsenore

This is a great enhancement, thank you!

The Carbon version works perfectly.
The Cocoa one does not have the focus ring any more but it still has the grey rectangle (a scroll bar?) at the right and the border is slightly different from the one of the other cells.

Anyway, this is definitely a great enhancement.

comment:4 Changed 2 years ago by vadz

  • Keywords Carbon removed
  • Milestone set to 3.0
  • Summary changed from wxGridCellTextEditor looks bad under OS X to wxGridCellTextEditor unusable in wxOSX/Cocoa

Apparently this is bad enough to make wxGrid unusable under wxOSX/Cocoa in practice.

comment:5 Changed 18 months ago by SC

(In [74100]) avoiding focus lost events when a different subview gets focused, see #14042

comment:6 Changed 18 months ago by csomor

  • Status changed from new to infoneeded_new

I don't have editing problems anymore, could you please test against trunk ?

comment:7 Changed 18 months ago by fsenore

  • Status changed from infoneeded_new to new

I tested the grid sample and now everything is working perfectly.
Thank you very much, this fixes a big problem for wxGrid users.

comment:8 Changed 18 months ago by csomor

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.