Skip to content

- trackbar handle width and height as parameters#54

Open
lukadt wants to merge 2 commits into
Dovyski:devfrom
lukadt:master
Open

- trackbar handle width and height as parameters#54
lukadt wants to merge 2 commits into
Dovyski:devfrom
lukadt:master

Conversation

@lukadt
Copy link
Copy Markdown

@lukadt lukadt commented Oct 12, 2018

I am using cvui in a touch device with a full hd resolution and being able to define trackbar handles is useful.

@Dovyski
Copy link
Copy Markdown
Owner

Dovyski commented Oct 13, 2018

Awesome, thank you!

In order to keep backward compatibility, would it be possible for you to move the theHandleWidth and theHandleHeight params to the end of the arguments list, please?

E.g.

bool trackbar(
  cv::Mat& theWhere,
  int theX,
  int theY,
  int theWidth,
  T *theValue,
  T theMin,
  T theMax,
  int theSegments = 1,
  const char *theLabelFormat = "%.1Lf",
  unsigned int theOptions = 0,
  T theDiscreteStep = 1,
  int theHandleWidth  =3,
  int theHandleHeight = 4);

@Dovyski Dovyski changed the base branch from master to dev April 8, 2020 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants