fix font rendering

[christian-rauch]

This reverts commit cba1ef2.

This fixes the font rendering issue: #940 (comment)

Fixes #939.

current master	PR

[christian-rauch]

@stevenlovegrove The "continuous coords" from the commit seem to blur the rendered text (#939). What was the reason for changing this? Can this be safely reverted or are we going to introduce issues somewhere else?

[mateosss]

Thanks for the update! I can confirm this fixes #939

[stevenlovegrove]

@christian-rauch looking at the commit, it seems I made an error and should have used ProjectionMatrixOrthographic(-0.5, w-0.5, -0.5, h-0.5, -1.0, 1.0).Load(); instead of the continuous coords.

The reason for the commit was to avoid the extra glMatrixMode calls (since we have to store the projection and model view matrices and then set them in a different method

Pangolin/components/pango_opengl/src/viewport.cpp

Line 64 in 13e4a18

void Viewport::ActivatePixelOrthographic() const

).

Reverting is safe. We could also replace the ProjectionMatrixOrthographic call with the correct pixel centric version to avoid the two extra GL API calls.

[christian-rauch]

@christian-rauch looking at the commit, it seems I made an error and should have used ProjectionMatrixOrthographic(-0.5, w-0.5, -0.5, h-0.5, -1.0, 1.0).Load(); instead of the continuous coords.

Thanks. This seems to work. I updated the PR.

@mateosss Can you check again with the new commit in the PR?

[mateosss]

Indeed this also fixes the issue, thank you both :)