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

Dev vue merge #103

Merged
merged 100 commits into from
Sep 14, 2022
Merged

Dev vue merge #103

merged 100 commits into from
Sep 14, 2022

Conversation

C0ntroller
Copy link
Member

Dieser PR führt die branches development und vue3 zusammen.
Zugehöriges issue: #102

Wenn angenommen: close #102

…t now and I don't know how to make it work, so like this it works
for settings card opening and more components
@OliEfr
Copy link
Member

OliEfr commented Sep 14, 2022

  • Wenn ich "Suchmaschinen Superkräfte" im On-Boarding deaktiviere, ist diese Funktion auf der Settings-Seite im Anschluss trotzdem aktiviert. Reload der Settings-Seite löst das Problem nicht.

  • Die Option "Suchmaschinen Superkräfte" auf der Settings-Seite funktioniert generell nicht: die Weiterleitung ist immer aktiviert, unabhängig von der Checkbox.

  • Ich habe das Problem, dass die Boxen über den Bildschirm hinausgehen, ich aber nicht scrollen kann --> Boxen kleiner oder Scrollbar?
    image

  • Kannst du den Text hier genau unter die Tick-Box alignen?
    image

  • Ich kann diese Funktion rechts aktivieren, ohne die Funktion links aktiviert zu haben und es kommt auch kein Berechtigungspopup. Intended?
    image

  • Kannst du die Checkbox standardmäßig aktivieren?
    image

  • Die Easteregg-Rakete ist nicht auswählbar, obwohl ich sie definitiv freigeschalten habe:
    image

  • Noch eine Kleinigkeit: Bitte nur probieren wenn es nicht zu kompliziert ist!. Hast du schonmal versucht für den Text unten generell Blocksatz zu machen (align=“justify“)? Ich glaube das könnte besser aussehen.
    image

@OliEfr
Copy link
Member

OliEfr commented Sep 14, 2022

@C0ntroller
image
Ich bekomme hier den Fehler "web_accessible_resources" und die Rakete wird nicht gezeigt.
Eigentlich ist ja aber /assets/* gelistet in web_accessible_resources..

Interessanterweise geht es nur mit der Easter-Egg-Rakete:
image

@Noxdor
Copy link
Member

Noxdor commented Sep 14, 2022

  • Kannst du den Text hier genau unter die Tick-Box alignen?
    image

Das kann ich nicht reproduzieren, bei mir sieht es so aus:
image

Wie ist es bei dir @C0ntroller?

@Noxdor
Copy link
Member

Noxdor commented Sep 14, 2022

Ah, ich merke gerade, dass es bei einer höheren Zoomstufe (dann wahrscheinlich auch kleinerem Screen) passiert. Das Wort "aktivieren" rutscht in die nächste Zeile und dann sieht es so aus.

@C0ntroller C0ntroller marked this pull request as ready for review September 14, 2022 11:24
@C0ntroller C0ntroller requested review from fugidev, OliEfr and a team as code owners September 14, 2022 11:24
Copy link
Member

@OliEfr OliEfr left a comment

Choose a reason for hiding this comment

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

Alles cool. Mergen.

@OliEfr OliEfr merged commit 89f9edc into develop Sep 14, 2022
@C0ntroller C0ntroller deleted the dev-vue-merge branch September 14, 2022 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request testing wanted Part of this pull request requires further testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TODOs for vue3 dashboard
3 participants