birkeh
/
o3de
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #1454 from aws-lumberyard-dev/LYN-3619

Browse Source 
New level dialog has no way to select a different folder
main
sphrose 5 years ago
parent b88d87a590
commit ef3e31c740
3 changed files with 245 additions and 105 deletions
Show Stats Download Patch File Download Diff File

151
Code/Sandbox/Editor/NewLevelDialog.cpp
Unescape Escape View File

@ -17,7 +17,10 @@
// Qt
#include <QtWidgets/QPushButton>
#include <QFileDialog>
#include <QMessageBox>
#include <QTimer>
#include <QToolButton>
// Editor
#include "NewTerrainDialog.h"
@ -30,11 +33,33 @@ AZ_POP_DISABLE_DLL_EXPORT_MEMBER_WARNING
// Folder in which levels are stored
static const char kNewLevelDialog_LevelsFolder[] = "Levels";
class LevelFolderValidator : public QValidator
{
public:
LevelFolderValidator(QObject* parent)
: QValidator(parent)
{
m_parentDialog = qobject_cast<CNewLevelDialog*>(parent);
}
QValidator::State validate([[maybe_unused]] QString& input, [[maybe_unused]] int& pos) const override
{
if (m_parentDialog->ValidateLevel())
{
return QValidator::Acceptable;
}
return QValidator::Intermediate;
}
private:
CNewLevelDialog* m_parentDialog;
};
// CNewLevelDialog dialog
CNewLevelDialog::CNewLevelDialog(QWidget* pParent /*=NULL*/)
: QDialog(pParent)
, m_ilevelFolders(0)
, m_bUpdate(false)
, ui(new Ui::CNewLevelDialog)
, m_initialized(false)
@ -43,46 +68,70 @@ CNewLevelDialog::CNewLevelDialog(QWidget* pParent /*=NULL*/)
setWindowFlags(windowFlags() & ~Qt::WindowContextHelpButtonHint);
setWindowTitle(tr("New Level"));
setMaximumSize(QSize(320, 280));
setMaximumSize(QSize(430, 180));
adjustSize();
// Default level folder is root (Levels/)
m_ilevelFolders = 0;
m_bIsResize = false;
ui->TITLE->setText(tr("Assign a name and location to the new level."));
ui->STATIC1->setText(tr("Location:"));
ui->STATIC2->setText(tr("Name:"));
// Level name only supports ASCII characters
QRegExp rx("[_a-zA-Z0-9-]+");
QValidator* validator = new QRegExpValidator(rx, this);
ui->LEVEL->setValidator(validator);
connect(ui->LEVEL_FOLDERS, SIGNAL(activated(int)), this, SLOT(OnCbnSelendokLevelFolders()));
validator = new LevelFolderValidator(this);
ui->LEVEL_FOLDERS->lineEdit()->setValidator(validator);
ui->LEVEL_FOLDERS->setErrorToolTip(
QString("The location must be a folder underneath the current project's %1 folder. (%2)")
.arg(kNewLevelDialog_LevelsFolder)
.arg(GetLevelsFolder()));
ui->LEVEL_FOLDERS->setClearButtonEnabled(true);
QToolButton* clearButton = AzQtComponents::LineEdit::getClearButton(ui->LEVEL_FOLDERS->lineEdit());
assert(clearButton);
connect(clearButton, &QToolButton::clicked, this, &CNewLevelDialog::OnClearButtonClicked);
connect(ui->LEVEL_FOLDERS->lineEdit(), &QLineEdit::textEdited, this, &CNewLevelDialog::OnLevelNameChange);
connect(ui->LEVEL_FOLDERS, &AzQtComponents::BrowseEdit::attachedButtonTriggered, this, &CNewLevelDialog::PopupAssetPicker);
connect(ui->LEVEL, &QLineEdit::textChanged, this, &CNewLevelDialog::OnLevelNameChange);
m_levelFolders = GetLevelsFolder();
m_level = "";
// First of all, keyboard focus is related to widget tab order, and the default tab order is based on the order in which
// widgets are constructed. Therefore, creating more widgets changes the keyboard focus. That is why setFocus() is called last.
// Secondly, using singleShot() allows setFocus() slot of the QLineEdit instance to be invoked right after the event system
// is ready to do so. Therefore, it is better to use singleShot() than directly call setFocus().
QTimer::singleShot(0, ui->LEVEL, SLOT(setFocus()));
QTimer::singleShot(0, ui->LEVEL, SLOT(OnStartup()));
ReloadLevelFolder();
}
CNewLevelDialog::~CNewLevelDialog()
{
}
void CNewLevelDialog::OnStartup()
{
UpdateData(false);
setFocus();
}
void CNewLevelDialog::UpdateData(bool fromUi)
{
if (fromUi)
{
m_level = ui->LEVEL->text();
m_levelFolders = ui->LEVEL_FOLDERS->currentText();
m_ilevelFolders = ui->LEVEL_FOLDERS->currentIndex();
m_levelFolders = ui->LEVEL_FOLDERS->text();
}
else
{
ui->LEVEL->setText(m_level);
ui->LEVEL_FOLDERS->setCurrentText(m_levelFolders);
ui->LEVEL_FOLDERS->setCurrentIndex(m_ilevelFolders);
ui->LEVEL_FOLDERS->lineEdit()->setText(m_levelFolders);
}
}
@ -90,7 +139,7 @@ void CNewLevelDialog::UpdateData(bool fromUi)
void CNewLevelDialog::OnInitDialog()
{
ReloadLevelFolders();
ReloadLevelFolder();
// Disable OK until some text is entered
if (QPushButton* button = ui->buttonBox->button(QDialogButtonBox::Ok))
@ -104,28 +153,19 @@ void CNewLevelDialog::OnInitDialog()
//////////////////////////////////////////////////////////////////////////
void CNewLevelDialog::ReloadLevelFolders()
void CNewLevelDialog::ReloadLevelFolder()
{
QString levelsFolder = QString(Path::GetEditingGameDataFolder().c_str()) + "/" + kNewLevelDialog_LevelsFolder;
m_itemFolders.clear();
ui->LEVEL_FOLDERS->clear();
ui->LEVEL_FOLDERS->addItem(QString(kNewLevelDialog_LevelsFolder) + '/');
ReloadLevelFoldersRec(levelsFolder);
ui->LEVEL_FOLDERS->lineEdit()->clear();
ui->LEVEL_FOLDERS->setText(QString(kNewLevelDialog_LevelsFolder) + '/');
}
//////////////////////////////////////////////////////////////////////////
void CNewLevelDialog::ReloadLevelFoldersRec(const QString& currentFolder)
QString CNewLevelDialog::GetLevelsFolder() const
{
QDir dir(currentFolder);
QDir projectDir = QDir(Path::GetEditingGameDataFolder().c_str());
QDir projectLevelsDir = QDir(QStringLiteral("%1/%2").arg(projectDir.absolutePath()).arg(kNewLevelDialog_LevelsFolder));
QFileInfoList infoList = dir.entryInfoList(QDir::Dirs | QDir::NoDotAndDotDot);
foreach(const QFileInfo &fi, infoList)
{
m_itemFolders.push_back(fi.baseName());
ui->LEVEL_FOLDERS->addItem(QString(kNewLevelDialog_LevelsFolder) + '/' + fi.baseName());
}
return projectLevelsDir.absolutePath();
}
//////////////////////////////////////////////////////////////////////////
@ -133,26 +173,47 @@ QString CNewLevelDialog::GetLevel() const
{
QString output = m_level;
if (m_itemFolders.size() > 0 && m_ilevelFolders > 0)
QDir projectLevelsDir = QDir(GetLevelsFolder());
if (!m_levelFolders.isEmpty())
{
output = m_itemFolders[m_ilevelFolders - 1] + "/" + m_level;
output = m_levelFolders + "/" + m_level;
}
return output;
QString relativePath = projectLevelsDir.relativeFilePath(output);
return relativePath;
}
//////////////////////////////////////////////////////////////////////////
void CNewLevelDialog::OnCbnSelendokLevelFolders()
bool CNewLevelDialog::ValidateLevel()
{
UpdateData();
// Check that the selected folder is in or below the project/LEVELS folder.
QDir projectLevelsDir = QDir(GetLevelsFolder());
QString selectedFolder = ui->LEVEL_FOLDERS->text();
QString absolutePath = QDir::cleanPath(projectLevelsDir.absoluteFilePath(selectedFolder));
QString relativePath = projectLevelsDir.relativeFilePath(absolutePath);
// Prevent saving to a different drive.
if (projectLevelsDir.absolutePath()[0] != absolutePath[0])
{
return false;
}
if (relativePath.startsWith(".."))
{
return false;
}
return true;
}
void CNewLevelDialog::OnLevelNameChange()
{
m_level = ui->LEVEL->text();
UpdateData(true);
// QRegExpValidator means the string will always be valid as long as it's not empty:
const bool valid = !m_level.isEmpty();
const bool valid = !m_level.isEmpty() && ValidateLevel();
// Use the validity to dynamically change the Ok button's enabled state
if (QPushButton* button = ui->buttonBox->button(QDialogButtonBox::Ok))
@ -161,6 +222,24 @@ void CNewLevelDialog::OnLevelNameChange()
}
}
void CNewLevelDialog::OnClearButtonClicked()
{
ui->LEVEL_FOLDERS->lineEdit()->setText(GetLevelsFolder());
UpdateData(true);
}
void CNewLevelDialog::PopupAssetPicker()
{
QString newPath = QFileDialog::getExistingDirectory(nullptr, QObject::tr("Choose Destination Folder"), GetLevelsFolder());
if (!newPath.isEmpty())
{
ui->LEVEL_FOLDERS->setText(newPath);
OnLevelNameChange();
}
}
//////////////////////////////////////////////////////////////////////////
void CNewLevelDialog::IsResize(bool bIsResize)
{

14
Code/Sandbox/Editor/NewLevelDialog.h
Unescape Escape View File

@ -34,6 +34,7 @@
#include <vector>
#include <QAbstractButton>
#include <QDialog>
#endif
@ -50,28 +51,29 @@ public:
CNewLevelDialog(QWidget* pParent = nullptr); // standard constructor
~CNewLevelDialog();
QString GetLevel() const;
void IsResize(bool bIsResize);
bool ValidateLevel();
protected:
void UpdateData(bool fromUi = true);
void OnInitDialog();
void ReloadLevelFolders();
void ReloadLevelFoldersRec(const QString& currentFolder);
void ReloadLevelFolder();
void showEvent(QShowEvent* event);
QString GetLevelsFolder() const;
protected slots:
void OnCbnSelendokLevelFolders();
void OnLevelNameChange();
void OnClearButtonClicked();
void PopupAssetPicker();
void OnStartup();
public:
QString m_level;
QString m_levelFolders;
int m_ilevelFolders;
bool m_bIsResize;
bool m_bUpdate;

185
Code/Sandbox/Editor/NewLevelDialog.ui
Unescape Escape View File

@ -6,74 +6,133 @@
<rect>
<x>0</x>
<y>0</y>
<width>320</width>
<height>280</height>
<width>430</width>
<height>180</height>
</rect>
</property>
<layout class="QGridLayout" name="gridLayout">
<item row="1" column="0" colspan="2">
<widget class="QGroupBox" name="PLACEHOLDER_TRN">
<layout class="QFormLayout" name="formLayout_2">
<property name="fieldGrowthPolicy">
<enum>QFormLayout::AllNonFixedFieldsGrow</enum>
</property>
</layout>
</widget>
</item>
<item row="0" column="0" colspan="2">
<widget class="QGroupBox" name="STATIC_GROUP1">
<property name="title">
<string>Level</string>
</property>
<layout class="QFormLayout" name="formLayout">
<item row="0" column="0">
<widget class="QLabel" name="STATIC2">
<property name="text">
<string>Name:</string>
</property>
<property name="alignment">
<set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignTop</set>
</property>
<property name="buddy">
<cstring>LEVEL</cstring>
</property>
<layout class="QVBoxLayout" name="verticalLayout">
<item>
<spacer name="topVerticalSpacer">
<property name="orientation">
<enum>Qt::Vertical</enum>
</property>
<property name="sizeHint" stdset="0">
<size>
<width>20</width>
<height>10</height>
</size>
</property>
</spacer>
</item>
<item>
<widget class="QLabel" name="TITLE">
<property name="text">
<string>Assign a name and location to the new level.</string>
</property>
<property name="alignment">
<set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignTop</set>
</property>
</widget>
</item>
<item row="0" column="1">
<widget class="QLineEdit" name="LEVEL">
<property name="alignment">
<set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignVCenter</set>
</property>
</item>
<item>
<spacer name="verticalSpacer">
<property name="orientation">
<enum>Qt::Vertical</enum>
</property>
<property name="sizeHint" stdset="0">
<size>
<width>20</width>
<height>20</height>
</size>
</property>
</spacer>
</item>
<item>
<layout class="QGridLayout" name="gridLayout">
<item row="0" column="0" colspan="2">
<widget class="QGroupBox" name="STATIC_GROUP1">
<property name="styleSheet">
<string notr="true">border: 0px;</string>
</property>
<layout class="QFormLayout" name="formLayout">
<item row="0" column="0">
<widget class="QLabel" name="STATIC2">
<property name="text">
<string>Name</string>
</property>
<property name="alignment">
<set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignTop</set>
</property>
<property name="buddy">
<cstring>LEVEL</cstring>
</property>
</widget>
</item>
<item row="0" column="1">
<widget class="QLineEdit" name="LEVEL">
<property name="alignment">
<set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignVCenter</set>
</property>
</widget>
</item>
<item row="1" column="0">
<widget class="QLabel" name="STATIC1">
<property name="minimumSize">
<size>
<width>100</width>
<height>0</height>
</size>
</property>
<property name="text">
<string>Location</string>
</property>
<property name="alignment">
<set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignTop</set>
</property>
<property name="buddy">
<cstring>LEVEL_FOLDERS</cstring>
</property>
</widget>
</item>
<item row="1" column="1">
<widget class="AzQtComponents::BrowseEdit" name="LEVEL_FOLDERS" native="true">
<property name="sizePolicy">
<sizepolicy hsizetype="MinimumExpanding" vsizetype="Preferred">
<horstretch>0</horstretch>
<verstretch>0</verstretch>
</sizepolicy>
</property>
</widget>
</item>
</layout>
</widget>
</item>
</layout>
</item>
<item>
<spacer name="verticalSpacer2">
<property name="orientation">
<enum>Qt::Vertical</enum>
</property>
</spacer>
</item>
<item row="1" column="0" colspan="2">
<widget class="QDialogButtonBox" name="buttonBox">
<property name="standardButtons">
<set>QDialogButtonBox::Cancel|QDialogButtonBox::Ok</set>
</property>
</widget>
</item>
<item row="1" column="0">
<widget class="QLabel" name="STATIC1">
<property name="text">
<string>Folder:</string>
</property>
<property name="alignment">
<set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignTop</set>
</property>
<property name="buddy">
<cstring>LEVEL_FOLDERS</cstring>
</property>
</widget>
</item>
<item row="1" column="1">
<widget class="QComboBox" name="LEVEL_FOLDERS"/>
</item>
</layout>
</widget>
</item>
<item row="2" column="0" colspan="2">
<widget class="QDialogButtonBox" name="buttonBox">
<property name="standardButtons">
<set>QDialogButtonBox::Cancel|QDialogButtonBox::Ok</set>
</property>
</widget>
</item>
</layout>
</item>
</layout>
</widget>
<customwidgets>
<customwidget>
<class>AzQtComponents::BrowseEdit</class>
<extends>QWidget</extends>
<header location="global">AzQtComponents/Components/Widgets/BrowseEdit.h</header>
<container>1</container>
</customwidget>
</customwidgets>
<resources/>
<connections>
<connection>

Write Preview
Loading…
Cancel
Save