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

[Hahmotelma] Lisää käyttöliittymä kaavamääräysryhmien luomiselle #51

Conversation

nmaarnio
Copy link
Collaborator

No description provided.

@nmaarnio nmaarnio force-pushed the 50-lisää-templaateista-irrallinen-käyttöliittymä-kaavamääräyksien-lisäämiselle branch from 392ba59 to 756f95d Compare November 7, 2024 10:15
@nmaarnio nmaarnio marked this pull request as ready for review November 7, 2024 15:15
@nmaarnio nmaarnio added the enhancement New feature or request label Nov 7, 2024
@nmaarnio nmaarnio requested review from Mtk112 and LKajan November 8, 2024 11:18
@nmaarnio nmaarnio changed the title 50 lisää templaateista irrallinen käyttöliittymä kaavamääräyksien lisäämiselle 50 lisää templaateista irrallinen käyttöliittymä kaavamääräysryhmien luomiselle Nov 8, 2024
@LKajan LKajan force-pushed the 50-lisää-templaateista-irrallinen-käyttöliittymä-kaavamääräyksien-lisäämiselle branch from e95fd16 to fd14767 Compare November 15, 2024 10:56
@LKajan LKajan changed the title 50 lisää templaateista irrallinen käyttöliittymä kaavamääräysryhmien luomiselle [Hahmotelma] Lisää käyttöliittymä kaavamääräysryhmien luomiselle Nov 15, 2024
Copy link
Contributor

@LKajan LKajan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hyvältä pohjalta näyttää!
Pari kommenttia lisäsin, mutta noita "lisäominaisuuksia" voidaan lisätä myöhemminkin. Niistä voisi sitten tehdä omat issuet.

def add_input_field(self, input_field_type: InputTypes):
if input_field_type == InputTypes.TEXT_VALUE:
widget = QTextEdit()
self.form_layout.addRow("Tekstiarvo", widget)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Olisi kiva, jos oikean muotoinen arvo-kenttä tulisi automaattisesti näkyviin valitun määräyslajin perusteella. Uskoisin, että yhdelle määräyslajille arvot ovat aina samanmuotoisia.
Alkuun tää voi ehkä olla näin ja toteuttaa tämä seuraavaan iteraatioon. Jos näin, niin tee tästä uusi issue.


def add_input_field(self, input_field_type: InputTypes):
if input_field_type == InputTypes.TEXT_VALUE:
widget = QTextEdit()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jos kyseessä tekstiarvo, niin tietomallin mukaan pitäisi syöttää myös tekstin "syntaksi" (en oo kyllä ihan varma mitä se meinaa).

widget = QTextEdit()
self.form_layout.addRow("Tekstiarvo", widget)
elif input_field_type == InputTypes.NUMERIC_VALUE:
widget = QgsDoubleSpinBox() # Or QgsSpinBox?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Numeromuotoiselle kentälle pitäisi syöttää myös tekstimuotoinen yksikkö.

widget = QgsDoubleSpinBox() # Or QgsSpinBox?
self.form_layout.addRow("Numeerinen arvo", widget)
elif input_field_type == InputTypes.CODE_VALUE:
widget = QComboBox()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

koodistotunnuksen lisäksi pitäisi kyetä syöttämään itse arvo ja näille jokin otsikko.

# TODO: widget.addItems()
self.form_layout.addRow("Koodiarvo", widget)

elif input_field_type == InputTypes.TEXT_INFO:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tietomallin mukaan lisätiedolla on aina "lisätiedon laji"-koodi, ja tämän lisäksi mahdollisesti jokin arvo.

Maakuntakaavaprojektin tietokannassa nämä on rakennettu tällä hetkellä vähän eri tavalla. Siinä kaavamääräyksellä on esim merkittävyys kenttä, johon valitaan lisätiedon laji-koodistosta jokin merkittävyyshierarkian koodi. Näin voisi periaatteessa tehdä sillä oletuksella, että samalla kaavakohteella ei tule montaa saman tyypin lisätietoa.

Näitä listätietojuttuja voitaisiin miettiä koko porukalla ja päättää miten me tämä toteutetaan.

@LKajan LKajan merged commit ca3b7c1 into main Nov 18, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lisää templaateista irrallinen käyttöliittymä kaavamääräysryhmien luomiselle
2 participants