Code review comment for lp://qastaging/~macslow/unity/unity.fix-924884

Revision history for this message
Andrea Azzarone (azzar1) wrote :

388 + // FIXME: A small hint for the keyboard-navigation highlight... you can use
389 + // the method...
390 + //
391 + // style.GetStarHighlightIcon()
392 + //
393 + // ... to get the correct texture from the style/theme and use an opacity of
394 + // .5 for blending it over the regular star-textures.
395 + //
396 + //GfxContext.QRP_1Tex(geo_star.x,
397 + // geo_star.y,
398 + // geo_star.width,
399 + // geo_star.height,
400 + // style.GetStarHighlightIcon()->GetDeviceTexture(),
401 + // texxform,
402 + // nux::Color(1.0f, 1.0f, 1.0f, 0.5f));
403 +
404 + geo_star.x += geo_star.width + star_gap;

Thanks for this :)

Code looks good.

review: Approve

« Back to merge proposal