Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LateNight :: vectorize remaining graphics #1530

Merged
merged 9 commits into from
Mar 12, 2018

Conversation

ronso0
Copy link
Member

@ronso0 ronso0 commented Mar 2, 2018

still working on it, don't merge

@ronso0
Copy link
Member Author

ronso0 commented Mar 2, 2018

@Be-ing @daschuer
Do you have any idea why there's no clipping indicator for Mic/Aux?
Are those level meters special in any way?

@daschuer
Copy link
Member

daschuer commented Mar 3, 2018

IMHO there should be a clipping indicator. It is currently missing somehow.

@ronso0
Copy link
Member Author

ronso0 commented Mar 3, 2018

IMHO there should be a clipping indicator. It is currently missing somehow.

Okay, I'll add it.

@ronso0
Copy link
Member Author

ronso0 commented Mar 3, 2018

Looking good now @200%, still fine @100%.
Please test on a real HD screen.

@Be-ing Be-ing added the skins label Mar 8, 2018
@Be-ing Be-ing added this to the 2.1.0 milestone Mar 8, 2018
@Be-ing
Copy link
Contributor

Be-ing commented Mar 8, 2018

The changes look sharp! However, it looks like the spinboxes are not scaling:

screenshot from 2018-03-07 18-56-09

@ronso0
Copy link
Member Author

ronso0 commented Mar 8, 2018

However, it looks like the spinboxes are not scaling

They still had min-/max-width in style.qss, changing it to width/height and even removing it doesn't help (master 2.2). So now neither the box nor the buttons have a size dfined anywhere, parent widgets provide enough space. IDK what's the matter..

@ronso0
Copy link
Member Author

ronso0 commented Mar 8, 2018

Stupid me,.. I was testing the above changes in the wrong skin folder.
It's working now: (don't know why the right-hand border is dropped when scaling up because of the negative right margin necessary to position the buttons on the edge)
latenight-beatbox-scaling-fix

If you want to fix it in #1518, the WBeatSpinbox section in style.qss should look like this:

WBeatSpinBox {
  /* make it 2px smaller in each dimension,
    border & padding will add to get the final size */
  width: 55px;
  height: 22px;
  border: 1px solid #666;
  /* size of spinbox: 54x22px
  incl. up/down button: 70x22px */
  border-radius: 2px;
  color: #cfb32c;
  background-color: #0f0f0f;
  selection-color: #0f0f0f;
  selection-background-color: #d2d2d2;
  font-size: 13px/13px;
  padding: 0px;
  padding: 1px 2px 1px 1px;
  margin: 0px -1px 2px 0px;
}
  WBeatSpinBox:hover {
    border-color: #888;
  }
  WBeatSpinBox:focus {
    border-color: #d2d2d2;
    color: #cfb32c;
  }

  WBeatSpinBox::up-button, WBeatSpinBox::down-button {
    subcontrol-origin: padding;
    position: relative;
    /* as with spinbox: border is added to size.
      define it with the same width as hover border, otherwise size jumps */
    width: 18px;
    height: 12px;
    margin: 0px 0px 0px 0px;
    padding: 0px;
    border: 0px;
    }

  WBeatSpinBox::up-button {
    subcontrol-position: top right;
    /* push it up 1px to cover the box' top border */
    margin-top: -1px;
    image: url(skin:/buttons/btn_beatbox_double.svg) no-repeat;
    }
    WBeatSpinBox::up-button:pressed {
      image: url(skin:/buttons/btn_beatbox_double_pressed.svg) no-repeat;
    }
  WBeatSpinBox::down-button {
    subcontrol-position: bottom right;
    /* push it down 1px to cover the box' top border */
    margin-bottom: -1px;
    image: url(skin:/buttons/btn_beatbox_halve.svg) no-repeat;
    }
    WBeatSpinBox::down-button:pressed {
      image: url(skin:/buttons/btn_beatbox_halve_pressed.svg) no-repeat;
    }

@nopeppermint
Copy link
Contributor

100% looks good (tested on Linux), 50% as well but the library is not scaling..

@ronso0
Copy link
Member Author

ronso0 commented Mar 8, 2018

but the library is not scaling

To properly test skin scaling with my non-HD screen I also had to maually set the system font size to double the regular value (xfce4), or use OS fontscaling (Ubuntu > Tweak Ubuntu app). this is because library components rely on the OS font size to scale (search box, tree view, library buttons etc.)

@ronso0
Copy link
Member Author

ronso0 commented Mar 12, 2018

@Be-ing

If you want to fix it in #1518, the WBeatSpinbox section in style.qss should look like this:

Sorry, somehow I was under the impression this was already merged..
Will add it here, of course.

@daschuer
Copy link
Member

Now the scaling works. Not perfect but OK. LGTM
@Be-ing: Can we merge this?

@Be-ing
Copy link
Contributor

Be-ing commented Mar 12, 2018

The space between the spinbox arrows looks odd, but I'm not sure if we can do better while using Qt4:
image

@Be-ing Be-ing merged commit 18bbf6b into mixxxdj:2.1 Mar 12, 2018
@ronso0 ronso0 deleted the LateNight-vectorize-graphics branch March 26, 2018 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants