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

20240205 feat floating flash messages #429

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

Conversation

z4m1n0
Copy link
Contributor

@z4m1n0 z4m1n0 commented Feb 6, 2025

Die CSS Klassen für das neue Design fehlt noch.

@rebootl Könntest du dir das mal anschauen?

  TODOs:
  - CSS für Design40 und Material anpassen
  - Fehler von t/flash_migration/deprecated_calls beheben
Erzeugt mit:

  find . -type f -exec sed -i '/^..-\? INCLUDE .common.flash.html. ..$/d'  {} \;
SL::YAML benutzt YAML::XS wenn es installisert ist
YAML::XS hat im Gegensatz zu YAML kein Stringify
@z4m1n0 z4m1n0 requested review from sschoeling and rebootl February 6, 2025 12:22
@rebootl
Copy link
Member

rebootl commented Feb 7, 2025

Hallo, ja ich schaue es mir an.
Vermutlich müsste ich einfach Klassen aus /css/kivitendo/main.css ins less übernehmen und die Farben mit Variablen ersetzen etc. Die Farben sind ja definiert, siehe z.B. /css/design40/manual/farben.html Eine frage wo habt ihr das jeweils getestet @z4m1n0 ? Ansonsten würde ich vielleicht mal noch schauen, dass ich dazu eine Test Seite machen kann in den Entwickler Optionen.
LG

@jbueren
Copy link
Contributor

jbueren commented Feb 10, 2025

Falls möglich keine Pull Requests erstellen, bei denen die Testfälle nicht durchlaufen

@rebootl
Copy link
Member

rebootl commented Feb 10, 2025

Hmm, Also ich wollte das jetzt mal testen und habe dazu den branch bei mir lokal ausgecheckt. Ich wollte dann ein Angebot erstellen, so dass ich allenfalls eine Flash Meldung erhalte. Da ist aber etwas kaputt. Wenn ich versuche einen Artikel hinzuzufügen unter "Artikel hier hinzufügen!", dann wird der Artikel nicht richtig übernommen und ich kann diesen auch nicht hinzufügen. Im Server log gibt es ein End of script output before headers: dispatcher.fcgi, referer: http://localhost:50110/kivitendo-erp/controller.pl?type=sales_quotation&action=Order%2fadd

Wenn ich auf den master Branch switche geht es. So kann ich da nicht wirklich viel machen...

Edit: Sehe da im log noch: mod_fcgid: stderr: Unsupported jQuery action: init__error at /usr/share/perl5/Rose/Object/MakeMethods/Generic.pm line 53., referer: http://localhost:50110/kivitendo-erp/controller.pl?type=sales_quotation&action=Order%2fadd

Verstehe aber ehrlich gesagt nicht ganz woher das kommt... Wie kommt denn das jQuery in die Rose lib.

Copy link
Member

Choose a reason for hiding this comment

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

Ist hier Möglicherweise zuviel raus gefallen, weil das alles auf einer Zeile steht? Denke es bräuchte das T8.

Copy link
Member

Choose a reason for hiding this comment

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

Hier auch? T8, L und LxERP.

Copy link
Member

@rebootl rebootl Feb 12, 2025

Choose a reason for hiding this comment

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

Hier auch. L, T8. Die Seite scheint zwar trotzdem zu funktionieren. Nein stimmt nicht die Seite ist kaputt. Ich war im neuen Design und das ist ja das alte.

@rebootl
Copy link
Member

rebootl commented Feb 12, 2025

Habe jetzt eine Test Seite erstellt mittels:

sub action_test_page {
  my ($self) = @_;

  flash('error', 'This is an error message');
  flash('warning', 'This is a warning message');
  flash('info', 'This is an info message');

  $self->render('flash/test_page');
}

Es kommt keine Fehlermeldung aber auch keine Flash Meldung.

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.

3 participants