NaviCube: improved style

[easyw-fc]

we could use a short version for labels:
TOP
BTM
FRNT
REAR
LFT
RX
SX
Still there is a need to add an uniform letter-spacing.

top.png (3.35 KiB) Viewed 1045 times

[wmayer]

The culprit of the bad look is this line:

Code: Select all

sansFont.setStretch(QFont::ExtraCondensed);

When commenting it out the text looks much better.

[tyszja]

The culprit of the bad look is this line:

Code: Select all

sansFont.setStretch(QFont::ExtraCondensed);

When commenting it out the text looks much better.

@wmayer: Thanks for your finding. Interesting... maybe its some kind of bug in Qt for Windows? Does the bottom layer label still fit within the face boundaries when it's commented out?

With this offending line I could squeeze in a larger font on a face while making the cube smaller and still keep the same readability level. But with this being broken on Windows we shouldn't merge this PR for sure! Thanks again for verifying this.
I might try the Windows build next time before making a pull request, but I don't have the toolchain set up on Windows yet. I won't be able to verify the macOS build though - not a mac owner.

[EDIT]
Since the PR was already merged, I did a fix on top of the current master branch to mitigate the character spacing error on Windows. Font size is now soft-constrained - has an upper limit to not overflow Cube's face. It's applied individually on each face only if it overflows due to the font size being to big. Currently it only makes the bottom face use slightly smaller font, while other faces are consistent in using slightly larger one - seems like a good compromise to me.
It's this branch: https://github.com/tyszja/FreeCAD/tree/naviCube_improve. But I still have to build it against Windows to verify.

BTW: I've did some work to make the Cube configurable, but it's still to early to post anything.

[alberich]

Cube could use one-letter labels with some renaming. One possibility is

• D dexter (Right)
  S sinister (Left)
  B bottom
  T top
  R rear
  F front

The dexter/sinister thing may be too obscure so maybe change Bottom to Underside, Rear to Back, giving

• R right
  L left
  U underside
  T top
  B back
  F front

[wmayer]

Thanks for your finding. Interesting... maybe its some kind of bug in Qt for Windows?

Possible. At least with Qt4.8 it looks good and there is no change required. With Qt5.6 or Qt5.12 the change is required.

Does the bottom layer label still fit within the face boundaries when it's commented out?

It's clearly outside of the cube's face. The letters "B" and "m" lie on the rounded edges of the cube.

I guess this has something to do that in Qt5.x the whole OpenGL support has been reworked. I remember that in the old QGLWidget class there was a method to paint text and with the workarounds for Qt5 the text looks really ugly.

For the Image module I once found a nice solution based on QPainterPath and the rendered text looks nice. See GLImageBox::renderText. Maybe the same idea (at least for Qt5 on Windows) could be applied on the NaviCube.

With this offending line I could squeeze in a larger font on a face while making the cube smaller and still keep the same readability level. But with this being broken on Windows we shouldn't merge this PR for sure! Thanks again for verifying this.
I might try the Windows build next time before making a pull request, but I don't have the toolchain set up on Windows yet. I won't be able to verify the macOS build though - not a mac owner.

I once had the situation that text painted on a GL canvas was truncated and after upgrading the graphic driver the issue was gone. So, it's possible that on your system it might look good but then on another computer it doesn't.

[tyszja]

Thanks for your finding. Interesting... maybe its some kind of bug in Qt for Windows?

Possible. At least with Qt4.8 it looks good and there is no change required. With Qt5.6 or Qt5.12 the change is required.

Does the bottom layer label still fit within the face boundaries when it's commented out?

It's clearly outside of the cube's face. The letters "B" and "m" lie on the rounded edges of the cube.

I guess this has something to do that in Qt5.x the whole OpenGL support has been reworked. I remember that in the old QGLWidget class there was a method to paint text and with the workarounds for Qt5 the text looks really ugly.

For the Image module I once found a nice solution based on QPainterPath and the rendered text looks nice. See GLImageBox::renderText. Maybe the same idea (at least for Qt5 on Windows) could be applied on the NaviCube.

With this offending line I could squeeze in a larger font on a face while making the cube smaller and still keep the same readability level. But with this being broken on Windows we shouldn't merge this PR for sure! Thanks again for verifying this.
I might try the Windows build next time before making a pull request, but I don't have the toolchain set up on Windows yet. I won't be able to verify the macOS build though - not a mac owner.

I once had the situation that text painted on a GL canvas was truncated and after upgrading the graphic driver the issue was gone. So, it's possible that on your system it might look good but then on another computer it doesn't.

Thanks. I'll see once I have all the build tools and libs set up on Windows. In the mean time I did a quick fix here: https://github.com/tyszja/FreeCAD/tree/naviCube_improve It removes the Condensed parameter and applies an upper size limit to a label only when it's to large to fit... this should now only affect the bottom face. As a result the bottom face should now use a slightly smaller font size than all the other use - it slightly breaks the consistency on this single face but shouldn't hurt your eyes too much.

[tyszja]

Hi,

If anyone cares, my NaviCube related branch has been updated with the configurable opacity, scale and position lock preferences (and hopefully fixed the unreadable text on Windows+Qt5).
The bad news is that I've probably messed up some translations as I couldn't just simply throw another few option widgets into this already mixed bag of options in the '3D View' section - so I did some spring cleaning there (It's a separate commit though thus can be skipped If anyone cares to pull without the cleanup).

Oh, the branch is here: https://github.com/tyszja/FreeCAD/tree/naviCube_improve

New and old look comparison attached. Left is the new one.

[wmayer]

New and old look comparison attached. Left is the new one

Please put all NaviCube settings into its own tab. The current 3D view page is already a bit overloaded and there shouldn't be added more widgets. Keep in mind that there are users with relative small displays and when a page has too many controls then it affects usability (if things work correctly then a scrollbar is shown, if not then the page is truncated).

[tyszja]

Keep in mind that there are users with relative small displays and when a page has too many controls then it affects usability (if things work correctly then a scrollbar is shown, if not then the page is truncated).

Right... When I looked at the current state of the 3D View tab with all these widgets mixed up with no sense I thought that no one thinks about the usability. Good I was wrong. I got your point, thanks.

[reox]

TOP
BTM
FRNT
REAR
LFT
RX
SX

seven labels for six faces?

Why not use N, E, S, W, T, B?

Code: Select all

    #                    North
    #
    #                +-----------+
    #               /           /|
    #              /  Top      / |
    #             /           /  |
    #            /           /   |
    #           +-----------+    |
    #           |           |    +
    #           |           |   /    East
    #   West    |   South   |  /
    #           |           | /
    #           |           |/
    #           +-----------+
    #   z
    #   ^          Bottom
    #   |   y
    #   | /
    #   |/
    #   +------> x

Or is the usage of cardinal directions not common in CAD? From what I'm used to, it is used in FEM to describe the faces of an hexahedron.