Mir

Code review comment for lp:~mir-team/mir/cursor-spike-phase-1-resubmit

  1. cursor-spike-phase-1-resubmit
  2. Merge into development-branch
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

If we're not messing with the abstraction that mg::Cursor represents then NullCursor makes sense. (I'm unconvinced by the naming as discussed above, but that isn't a blocker for this MP.)

I tend to agree with Daniel about "theme_name" - we don't need any representation of themes (yet).

Is there anything cursor specific about CursorImage? Or is it just a holder for any ARGB image?

review: Needs Fixing

« Back to merge proposal