Option to show menu on hover about material-decoration HOT 18 CLOSED

I've merged this feature and titlebar alignment to the master branch.

from material-decoration.

The menuonhover branch has this feature. I now need to add a config option.

I need to figure out how I want to handle the caption. It currently "dodges" the menu which isn't needed when the menu is hidden.

I should probably do a comprehensive alignment option like Breeze.

• Left/Center/Right
• Center [between left/right buttons]
• Center (Full Width) [true center of window]
• Center (Full Width + Dodge Menu) [current MaterialDecoration behavior]

from material-decoration.

Hmm. According to the Unity video https://youtu.be/nzFXRz-b3Ns?t=40

• Captions are left aligned
• Menu doesn't appear when hovered until the window is focused
• The title completely disappears, theres' no gradient (at least not in Ubuntu 14.04).

from material-decoration.

@nicman23

hey adding to this, could we have the title of the window instead of empty space?
so on no hover -> title
on hover -> menu

I implement this feature you want. you can compile and test it.
https://github.com/mojtaba82/material-decoration

from material-decoration.

Thanks for reporting the bug @nicman23. I fixed that bug and made the titlebar repaint when toggling always show.

from material-decoration.

To reproduce it, you need to click at points sit at the lower-half part of buttons...

I cinfirm this.

from material-decoration.

I already have background opacity animations, so making a reveal on hover animation isn't too difficult. The hard part is having the entire menu drawn in sync as the current bg hover animation is per button. I will need another QAnimation, probably in AppMenuButtonGroup. I'll also need to rename the existing opacity property to backgroundOpacity, or hoverTween as the foreground color was also mixed on hover.

This code snippet will make all buttons transparent, and reveal just 1 button on hover.

QColor AppMenuButton::foregroundColor() const
{
    // ...
    } else {
        QColor hoverColor = Button::foregroundColor();
        QColor normalColor = QColor(hoverColor);
        normalColor.setAlphaF(0);

        return KColorUtils::mix(
            normalColor,
            hoverColor,
            opacity());
    }
}

I also need to determine if the menu should be revealed if the caption area is hovered, or if anywhere in the titlebar is hovered. If the latter, then the hover events need to be handled in Decoration. I can still keep the new opacity property in AppMenuButtonGroup though.

• https://doc.qt.io/qt-5/qpainter.html
• https://www.youtube.com/watch?v=teAFgtHWDso

from material-decoration.

can we have this without any animation?

from material-decoration.

I'm also interested on this, even without animations

from material-decoration.

I fork this nice decoration. and add a branch named captionmenufeatcher and merge caption and menu like unity lim.
It work but I'm not professional in programming. I would be glad if you test it.
Thanks

from material-decoration.

hey adding to this, could we have the title of the window instead of empty space?

so on no hover -> title

on hover -> menu

from material-decoration.

• Caption will now hide on hover, and it's position ignores the space taken up by the menu.
• There's now a toggle in the config.
  • I used a hack to treat the "checkbox" as 2 radio buttons so that the on/off state has descriptive labels.
  • I need to bind the config toggle to redraw the menu.

from material-decoration.

small bug report, the menu is visible on window creation not the title

from material-decoration.

Is this a related bug?

from material-decoration.

I have same problem occasionally.

from material-decoration.

The hover animation isn't checking if the menu is open atm. I did experience that once, but it rarely happens with Breeze's contextmenu.

Looks like I need to check (m_currentIndex != -1) in the updateOpacity func.

setOpacity(m_alwaysShow || m_hovered ? 1 : 0);

from material-decoration.

To reproduce it, you need to click at points sit at the lower-half part of buttons...

from material-decoration.

Should be fixed with d026781 and 8873774.

Thanks @trmdi for your PR #34. I expanded on it to add a bool showing property which represents the alwaysShow || hovered || isMenuOpen().

from material-decoration.