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

Deduplicate screen architecture #136

Open
1 task
FSadrieh opened this issue Dec 9, 2022 · 4 comments · May be fixed by #264
Open
1 task

Deduplicate screen architecture #136

FSadrieh opened this issue Dec 9, 2022 · 4 comments · May be fixed by #264
Assignees

Comments

@FSadrieh
Copy link
Contributor

FSadrieh commented Dec 9, 2022

As a developer, I do not want to write everything twice for both screens.

Acceptance Criteria

  • Components with ID don't need to be replicated

For questions regarding this issue, reach out to: @ko-redtruck

@FSadrieh FSadrieh moved this from 📋 PO WIP to 📋 Product Backlog in Bookkeeper Portal – 🟥 Edition Dec 9, 2022
@codingWhale13
Copy link
Contributor

codingWhale13 commented Dec 9, 2022

Das bezieht sich vor allem auf die Dateien _item_smallscreen.html.erb und _item_largescreen.html.erb, die beide in app/views/groups/show.html.erb gerendert werden.

@DefineDan
Copy link
Collaborator

@konrad-gerlach @janiswehen bezieht sich das auf das, was ihr initial gebaut habt?

@konrad-gerlach
Copy link
Collaborator

Das stimmt. Wir haben mehrere verschiedene Sachen ausprobiert, z.B.: User Agent Tests und mit Bootstrap Media Queries umgehen und das schien die beste Möglichkeit zu sein möglichst einfach (d.h. ohne super kompliziertes und schlecht wartbares JS Umrendern) die dynamische Anpassung an die unterschiedlichen Bildschirmgrößen zu gewährleisten. Das Problem ist, dass sich die Wireframes so stark im DOM unterschieden haben, dass wir die nicht einfach mit ein biscchen CSS und Bootstrap anpassen konnten.

@konrad-gerlach
Copy link
Collaborator

Was gemacht werden könnte ist, dass wir mit JS die Bildschirmgröße abfragen und je nachdem nur eine der beiden Dateien rendern. Ich sehe nur nicht, wie wir um das Problem beider Dateien herumkommen.

@FSadrieh FSadrieh moved this from 📋 Product Backlog to 🔖 TODO in Bookkeeper Portal – 🟥 Edition Jan 6, 2023
@navetis navetis self-assigned this Jan 6, 2023
@navetis navetis moved this from 🔖 TODO to 🏗 In progress: Ongoing in Bookkeeper Portal – 🟥 Edition Jan 6, 2023
@FSadrieh FSadrieh moved this from 🏗️ In progress: Ongoing to 🏗️ In progress: Done in Bookkeeper Portal – 🟥 Edition Jan 15, 2023
@FSadrieh FSadrieh moved this from 🏗️ In progress: Done to 👀 In review in Bookkeeper Portal – 🟥 Edition Jan 16, 2023
@FSadrieh FSadrieh moved this from 👀 In review to 🛑 Blocked in Bookkeeper Portal – 🟥 Edition Jan 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🛑 Blocked
5 participants