From patchwork Fri Aug 12 12:18:35 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Utkarsh Tiwari X-Patchwork-Id: 17095 Return-Path: X-Original-To: parsemail@patchwork.libcamera.org Delivered-To: parsemail@patchwork.libcamera.org Received: from lancelot.ideasonboard.com (lancelot.ideasonboard.com [92.243.16.209]) by patchwork.libcamera.org (Postfix) with ESMTPS id 615A8C3272 for ; Fri, 12 Aug 2022 12:18:44 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id B91D06332B; Fri, 12 Aug 2022 14:18:43 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org; s=mail; t=1660306723; bh=9hfG2KEdD9kiPpB7ndMV2msmDt68GxcRdiQOPih0CgU=; h=To:Date:In-Reply-To:References:Subject:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To: From; b=wY3ldbbTnC5Our9VJLoY60vCkYqUnFHSXDJek9ezEwQdkyzbVWvSlmGL9ncO9qpLN B830T27Dfj9zal4vk1MKwvKwkwSh+2SPANizu32OUjUNHtEL3ikZQK4u2IoSGnP+kA 2fAkzpywBIVmu4HbXJeMLqFc3RDsyWVXPOeAhLN4fg4Gbt43oqNs2ny0B2+fAKj3eA IBg7CUfonrkkCxGyi0S2ACXX+cZ9QZpb+Q+t7zLokujbRIzN6o9wxMF7R14F99dB2E F3BKmPDI4h3PxYDMxpZHf71ZLvdEZ8wGih26XmtqiWU/JiyU1itTACEdWuauqKpmXg 2M+761KkA9S0Q== Received: from mail-pj1-x102b.google.com (mail-pj1-x102b.google.com [IPv6:2607:f8b0:4864:20::102b]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 0305E603EA for ; Fri, 12 Aug 2022 14:18:42 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="DPpG6TQp"; dkim-atps=neutral Received: by mail-pj1-x102b.google.com with SMTP id s5-20020a17090a13c500b001f4da9ffe5fso8161285pjf.5 for ; Fri, 12 Aug 2022 05:18:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc; bh=K5JhIiFsYkZAHageL2WWC65VHNYg80GNXXE6Me+3AFo=; b=DPpG6TQpsfhkMmiFOg4nuqQI5HjrJeS85Uh/BDakO27tBTdFQ2rn+hGowdRAQtbrWe Tl2Jm0Jt97q6VYyphQoKdiY9JAWS+qoK7Hie3EGrLcgnRb3HFxnBhEhRpgGf5HaYLW8C FB03XORd9+87SLIIfX1P5Duh9Ul9R6cCumevpyeKVXfNlniwfCBRkr0nRBll3pXVyiyR Lnwjs536TpFGh7uoOoHsn2Wt0XREFFT0V9y0Jr6Cl4RMi9BpW0uj6M8Orx52mviyZZ5Z VO3FH9OIojmSVXRksLrd7AzhJsq7BKUPGm6j5i6pBtQJoPDV+NH7jqJFj/Qyion8/wb+ uM1Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc; bh=K5JhIiFsYkZAHageL2WWC65VHNYg80GNXXE6Me+3AFo=; b=BaO9gCJl7VsyQcFzGs4zDllS2fqAglqQLpz5sEScH7Ny7Q72W0W+4kbJoh2iVPHR29 j8C7g3M77cwi+OKKi+imcWhTvWGBfNV/SOWYOSWEqoeZZxklhiQaWu1TDhGVQXqYo+iP 4EjgUnLOLgov6maddcYrUroDnqJaZmPXrNAVYstptkqbXLybsWQ0JqtHZMnohAv881d1 G0P3CHQKoF8kMYyQPja+Lz586x02I32j7TpXIv+NNLdj+9RWxpkcxSZrZiCBf2Y8s3V1 1b4p55Ka5Cl6pOYtWpte0U1KF35fnWztoHL5XqSYofSd+Zl1nQEzVtSy9gMWA20qhkGL WRkQ== X-Gm-Message-State: ACgBeo1vKAjsX8Gj3gpPd3elpbdSw7qCYQ6EnljdNmADo0MCqYOJDRqE vQ1ZQEVf37mima++0S7WVSP+eL0XdqE= X-Google-Smtp-Source: AA6agR5yt82Xa2EmP4wcEiCiyQW3s3+jjO5r7ZirTa/ikL19qyumKJeCYUcq36O6qzme1b+rVAUwzA== X-Received: by 2002:a17:902:e5cd:b0:16d:b92c:6848 with SMTP id u13-20020a170902e5cd00b0016db92c6848mr3710197plf.56.1660306721321; Fri, 12 Aug 2022 05:18:41 -0700 (PDT) Received: from localhost.localdomain ([2404:bd00:3:dc0d:d8e:96a2:2dbe:5a83]) by smtp.gmail.com with ESMTPSA id h26-20020aa79f5a000000b0052d33bf14d6sm1472727pfr.63.2022.08.12.05.18.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 12 Aug 2022 05:18:40 -0700 (PDT) To: libcamera-devel@lists.libcamera.org Date: Fri, 12 Aug 2022 17:48:35 +0530 Message-Id: <20220812121835.21974-1-utkarsh02t@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220810150349.414043-8-utkarsh02t@gmail.com> References: <20220810150349.414043-8-utkarsh02t@gmail.com> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v8.1 7/8] qcam: CamSelectDialog: Display Capture script path X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: Utkarsh Tiwari via libcamera-devel From: Utkarsh Tiwari Reply-To: Utkarsh Tiwari Errors-To: libcamera-devel-bounces@lists.libcamera.org Sender: "libcamera-devel" Display the path of the selected capture script in a thinner font. Signed-off-by: Utkarsh Tiwari --- Difference from v8: 1. scriptPathLabel_ now removed when informScriptReset() is invoked. Difference from v7: 1. The scriptPathLabel_ has a fixed parent now captureWidget src/qcam/cam_select_dialog.cpp | 40 ++++++++++++++++++++++++++++------ src/qcam/cam_select_dialog.h | 7 +++++- src/qcam/main_window.cpp | 3 ++- 3 files changed, 41 insertions(+), 9 deletions(-) diff --git a/src/qcam/cam_select_dialog.cpp b/src/qcam/cam_select_dialog.cpp index 58cbfc28..44fa3ce6 100644 --- a/src/qcam/cam_select_dialog.cpp +++ b/src/qcam/cam_select_dialog.cpp @@ -19,10 +19,12 @@ #include #include #include +#include CameraSelectorDialog::CameraSelectorDialog(libcamera::CameraManager *cameraManager, - bool isScriptRunning, QWidget *parent) - : QDialog(parent), cm_(cameraManager), isScriptRunning_(isScriptRunning) + bool isScriptRunning, std::string scriptPath, QWidget *parent) + : QDialog(parent), cm_(cameraManager), + isScriptRunning_(isScriptRunning), scriptPath_(scriptPath) { /* Use a QFormLayout for the dialog. */ QFormLayout *layout = new QFormLayout(this); @@ -40,14 +42,31 @@ CameraSelectorDialog::CameraSelectorDialog(libcamera::CameraManager *cameraManag connect(cameraIdComboBox_, &QComboBox::currentTextChanged, this, &CameraSelectorDialog::handleCameraChange); + /* Setup widget for capture script button. */ + QWidget *captureWidget = new QWidget; + captureWidgetLayout_ = new QVBoxLayout(captureWidget); + captureWidgetLayout_->setMargin(0); + captureScriptButton_ = new QPushButton; connect(captureScriptButton_, &QPushButton::clicked, this, &CameraSelectorDialog::handleCaptureScriptButton); + captureWidgetLayout_->addWidget(captureScriptButton_); + + /* Use a thinner font to indicate script info. */ + QFont smallFont; + smallFont.setWeight(QFont::Thin); + + scriptPathLabel_ = new QLabel(captureWidget); + scriptPathLabel_->setFont(smallFont); + scriptPathLabel_->setWordWrap(true); /* Display the action that would be performed when button is clicked. */ - if (isScriptRunning_) + if (isScriptRunning_) { captureScriptButton_->setText("Stop"); - else + + scriptPathLabel_->setText(QString::fromStdString(scriptPath_)); + captureWidgetLayout_->addWidget(scriptPathLabel_); + } else captureScriptButton_->setText("Open"); /* Setup the QDialogButton Box */ @@ -64,7 +83,7 @@ CameraSelectorDialog::CameraSelectorDialog(libcamera::CameraManager *cameraManag layout->addRow("Camera:", cameraIdComboBox_); layout->addRow("Location:", cameraLocation_); layout->addRow("Model:", cameraModel_); - layout->addRow("Capture Script:", captureScriptButton_); + layout->addRow("Capture Script:", captureWidget); layout->addWidget(buttonBox); } @@ -139,16 +158,22 @@ void CameraSelectorDialog::handleCaptureScriptButton() Q_EMIT stopCaptureScript(); isScriptRunning_ = false; captureScriptButton_->setText("Open"); + + captureWidgetLayout_->removeWidget(scriptPathLabel_); } else { selectedScriptPath_ = QFileDialog::getOpenFileName(this, "Run Capture Script", QDir::currentPath(), "Capture Script (*.yaml)") .toStdString(); - if (!selectedScriptPath_.empty()) + if (!selectedScriptPath_.empty()) { captureScriptButton_->setText("Loaded"); - else + scriptPathLabel_->setText(QString::fromStdString(selectedScriptPath_)); + captureWidgetLayout_->addWidget(scriptPathLabel_); + } else { captureScriptButton_->setText("Open"); + captureWidgetLayout_->removeWidget(scriptPathLabel_); + } } } @@ -170,6 +195,7 @@ void CameraSelectorDialog::informScriptReset() isScriptRunning_ = false; scriptPath_.clear(); selectedScriptPath_.clear(); + captureWidgetLayout_->removeWidget(scriptPathLabel_); captureScriptButton_->setText("Open"); } diff --git a/src/qcam/cam_select_dialog.h b/src/qcam/cam_select_dialog.h index bbdf897e..72dfbb14 100644 --- a/src/qcam/cam_select_dialog.h +++ b/src/qcam/cam_select_dialog.h @@ -18,13 +18,15 @@ #include #include #include +#include +#include class CameraSelectorDialog : public QDialog { Q_OBJECT public: CameraSelectorDialog(libcamera::CameraManager *cameraManager, - bool isScriptRunning, QWidget *parent); + bool isScriptRunning, std::string scriptPath, QWidget *parent); ~CameraSelectorDialog() = default; @@ -62,5 +64,8 @@ private: QComboBox *cameraIdComboBox_; QLabel *cameraLocation_; QLabel *cameraModel_; + + QVBoxLayout *captureWidgetLayout_; QPushButton *captureScriptButton_; + QLabel *scriptPathLabel_; }; diff --git a/src/qcam/main_window.cpp b/src/qcam/main_window.cpp index 3c7c3173..753e1af9 100644 --- a/src/qcam/main_window.cpp +++ b/src/qcam/main_window.cpp @@ -339,7 +339,8 @@ std::string MainWindow::chooseCamera() /* Construct the selection dialog, only the first time. */ if (!cameraSelectorDialog_) - cameraSelectorDialog_ = new CameraSelectorDialog(cm_, scriptRunning, this); + cameraSelectorDialog_ = new CameraSelectorDialog(cm_, scriptRunning, + scriptPath_, this); connect(cameraSelectorDialog_, &CameraSelectorDialog::stopCaptureScript, this, &MainWindow::stopCaptureScript);