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

2024 item input below item list #408

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

sschoeling
Copy link
Member

@sschoeling sschoeling commented Nov 20, 2024

Dieser PR enthält 3 Features im Auftrag von Jan.

Ziel ist, dass Benutzer die die alte Auftragsmaske gewohnt sind sich ein wenig Look&Feel davon zurückkonfigurieren können.

  1. In der Auftragsmaske kann die Artikeleingabe jetzt unter die Positionen konfiguriert werden. Das interne flag dazu heisst "order_item_input_position", und ist sowohl in der Mandantenkonfig (unter Features -> Einkauf & Verkauf) als auch in den Benutzereinstellungen (Anzeigeoptionen) zu finden. Benutzereinstellungen überlagern Mandantenkonfiguration. Ich habe die Benutzerkofiguration mit Bernds SL::UserPreferences::* gemacht, schau bitte mal ob das so okay ist Bernd.

  2. Um die Eingabe flüssiger zu machen, ist das Scrollverhalten in der Auftragsmaske jetzt leicht anders: Wenn der Scrollbereich für die Positionen ausgestellt ist (Benutzereinstellungen -> Prozentuale Höhe der Positionen = 0) und die Artikeleingabe unter den Positionen gerendert wird, und die Seite groß genug wird, dass es Scrollbars gibt, scrollt die Eingabe so, dass die vertikale Position der Artikeleingabe konstant bleibt - die Seite rutscht also nach oben.

  3. Weil die Artikeleingabe durch diese Änderungen sehr weit unten auf der Seite ist, habe ich dem Part Picker Verhalten gegeben, dass das autocomplete Menü über der Eingabe gerendert wird wenn unter dem input nicht genug Platz ist, aber über dem input genug Platz ist. Die Positionierung ist sticky, wird also nur neu entschieden, wenn das autocomplete zu und neu aufgemacht wird, nicht beim neudimensionieren wenn sich durch Eingabe die Auswahl ändert.

Option ist in der Mandatenkonfiguration unter Features -> Einkauf &
Verkauf -> Position der Artikeleingabe in Aufträgen und Angeboten
…e ohne Scrollbereich

Wenn die Artikeleingabe unten gerendert wird, und die Positionsliste
nicht begrenzt ist, gilt jetzt folgendes Verhalten:

Wenn nach dem Einfügen der neuen Zeile die Seite scrollbar ist, und
durch scrollen die Eingabeleite auf konstanter Höhe gehalten werden
kann, dann wird so gescrollt.
@rebootl
Copy link
Member

rebootl commented Dec 5, 2024

Hi

Hier ist mir aufgefallen, dass noch die Template Anpassungen fürs neue Design 4.0 fehlen. templates/design40_webpages/

Zur zeit erstellen wir ja jeweils noch beide Varianten. Vielleicht in der nächsten Version (falls 4.0) dann nur noch die neuen.
Denke diese müssten noch ergänzt werden.

LG
Cem

Im Gegensatz zum alten template Satz ist hier die Artikeleingabe nicht
im scrollbaren Bereich enthalten, weil der sowieso schon recht klein
ist.
@sschoeling
Copy link
Member Author

Hier ist mir aufgefallen, dass noch die Template Anpassungen fürs neue Design 4.0 fehlen. templates/design40_webpages/

Zur zeit erstellen wir ja jeweils noch beide Varianten. Vielleicht in der nächsten Version (falls 4.0) dann nur noch die neuen. Denke diese müssten noch ergänzt werden.

Moin Cem,

ist korrekt, ist jetzt nachgepflegt.

Das Verhalten ist hier jetzt aber ein wenig anders. Da im design4.0 der Scrollbereich sowieso sehr klein ist, habe ich die Artikeleingabe darunter außerhalb der Itemliste gepackt. Teste mal bitte ob sich das so flüssig anfühlt.

@sschoeling sschoeling requested a review from rebootl January 24, 2025 14:32
@rebootl
Copy link
Member

rebootl commented Jan 27, 2025

Ja sieht gut aus von mir aus. 👍 Vielleicht am besten noch mit den Partnern besprechen die sich das Feature gewünscht haben. Vielleicht am nächsten Partnern Treff zum Beispiel.
LG
Cem

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