Mobile improvements #3803

Merged

hansegucker merged 15 commits from mobile-improvements into master 2026-02-20 16:34:23 +01:00

Conversation 23 Commits 15 Files changed 8 +128 -76

leuckeju commented 2026-02-15 22:06:11 +01:00

Member

Copy link

Information and screenshot

Closes #1564

No further information have been provided.

Checklist

• tox -e reformat,lint used
• Changelog updated/No update necessary
• Documentation (incl. Screenshots) up-to-date
• Copyright in apps.py and README.rst up-to-date
• (Unit) tests for new feature/regression tests for bug fixes (only if useful)
• Adhere to URL scheme
• Tested with admin account
• Tested with normal account (based on permissions)

Backend checklist

• Use permissions for all GraphQL mutations
• Use query optimizer for all GraphQL queries
• Don't use inefficient DB queries

Frontend checklist (mandatory with ~"part::frontend")

• Screenshots are attached (mobile and desktop)
• Tested on mobile, tablet, and desktop sizes
• Adhered to frontend guidelines
• Checked with Firefox Accessibility Checker

## Information and screenshot Closes #1564 No further information have been provided. ## Checklist - [x] `tox -e reformat,lint` used - [x] Changelog updated/No update necessary - [ ] Documentation (incl. Screenshots) up-to-date - [ ] Copyright in `apps.py` and `README.rst` up-to-date - [ ] (Unit) tests for new feature/regression tests for bug fixes (only if useful) - [ ] Adhere to [URL scheme](https://edugit.org/AlekSIS/official/AlekSIS/-/issues/86) - [ ] Tested with admin account - [ ] Tested with normal account (based on permissions) ## Backend checklist - [ ] Use permissions for all GraphQL mutations - [ ] Use query optimizer for all GraphQL queries - [ ] Don't use inefficient DB queries ## Frontend checklist (mandatory with ~"part::frontend") - [ ] Screenshots are attached (mobile and desktop) - [x] Tested on mobile, tablet, and desktop sizes - [ ] Adhered to [frontend guidelines](https://edugit.org/AlekSIS/official/AlekSIS/-/blob/master/project/frontend_guidelines.rst?ref_type=heads) - [ ] Checked with [Firefox Accessibility Checker](https://firefox-source-docs.mozilla.org/devtools-user/accessibility_inspector/)    

leuckeju commented 2026-02-15 22:12:50 +01:00

Author

Member

Copy link

added 1 commit

• b692ac24 - Reformat

Compare with previous version

added 1 commit <ul><li>b692ac24 - Reformat</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2158/diffs?diff_id=83430&start_sha=95af696d3697e7187334da840293bca997ce385d)

leuckeju commented 2026-02-17 17:25:17 +01:00

Author

Member

Copy link

marked the checklist item tox -e reformat,lint used as completed

marked the checklist item **`tox -e reformat,lint` used** as completed

leuckeju commented 2026-02-17 17:25:24 +01:00

Author

Member

Copy link

marked the checklist item Tested on mobile, tablet, and desktop sizes as completed

marked the checklist item **Tested on mobile, tablet, and desktop sizes** as completed

leuckeju commented 2026-02-17 17:40:10 +01:00

Author

Member

Copy link

added 1 commit

• 598dd810 - Update changelog

Compare with previous version

added 1 commit <ul><li>598dd810 - Update changelog</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2158/diffs?diff_id=83473&start_sha=b692ac24e99fa57ee406c297d8cadb6be8a9a589)

leuckeju commented 2026-02-17 17:51:52 +01:00

Author

Member

Copy link

marked the checklist item Changelog updated/No update necessary as completed

marked the checklist item **Changelog updated/No update necessary** as completed

hansegucker commented 2026-02-17 20:36:24 +01:00

Owner

Copy link

And now the screenshot ...

And now the screenshot ...

leuckeju commented 2026-02-18 10:15:56 +01:00

Author

Member

Copy link

added 1 commit

• a9b87013 - Increase spacing after field

Compare with previous version

added 1 commit <ul><li>a9b87013 - Increase spacing after field</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2158/diffs?diff_id=83499&start_sha=598dd810e709f82459288dee0a54c2901fe24a77)

leuckeju commented 2026-02-18 10:16:46 +01:00

Author

Member

Copy link

changed the description

changed the description

leuckeju commented 2026-02-18 10:16:47 +01:00

Author

Member

Copy link

assigned to @hansegucker

assigned to @hansegucker

leuckeju commented 2026-02-18 10:19:20 +01:00

Author

Member

Copy link

added 69 commits

• a9b87013...ea30b748 - 68 commits from branch master
• 2851e9fc - Merge branch 'master' into 'mobile-improvements'

Compare with previous version

added 69 commits <ul><li>a9b87013...ea30b748 - 68 commits from branch <code>master</code></li><li>2851e9fc - Merge branch &#39;master&#39; into &#39;mobile-improvements&#39;</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2158/diffs?diff_id=83500&start_sha=a9b87013939034648ed8ebb3a0a279622cfd376c)

leuckeju commented 2026-02-18 10:25:30 +01:00

Author

Member

Copy link

did so

did so

hansegucker commented 2026-02-18 11:30:07 +01:00

Owner

Copy link

resolved all threads

resolved all threads

hansegucker commented 2026-02-18 11:33:23 +01:00

Owner

Copy link

Can we remove the indention for completed todos? I don't think that it's useful and it consumes too much space.

I am also not sure about the close button in the calendar view. It probably should at least get an icon that most users connect with closing something.

Also the "Completed" text above the range slider doesn't align properly, that doesn't look too well.

And maybe you could add some optimization to the datetime field? I think the date part should consume more place and maybe we show the clock icon only if it's a time-only field?

Can we remove the indention for completed todos? I don't think that it's useful and it consumes too much space. I am also not sure about the close button in the calendar view. It probably should at least get an icon that most users connect with closing something. Also the "Completed" text above the range slider doesn't align properly, that doesn't look too well. And maybe you could add some optimization to the datetime field? I think the date part should consume more place and maybe we show the clock icon only if it's a time-only field?

leuckeju commented 2026-02-18 13:33:48 +01:00

Author

Member

Copy link

resolved all threads

resolved all threads

leuckeju commented 2026-02-18 20:12:52 +01:00

Author

Member

Copy link

marked this merge request as draft

marked this merge request as **draft**

leuckeju commented 2026-02-18 23:27:21 +01:00

Author

Member

Copy link

• indentation removed
• Close button changed as discussed live
• "Completed" text was aligned
• Icons optimized, but sadly not able to increase datefield size, so on (very) small displays it will now look like this:
  

- [x] indentation removed - [x] Close button changed as discussed live - [x] "Completed" text was aligned - [ ] Icons optimized, but sadly not able to increase datefield size, so on (very) small displays it will now look like this: 

leuckeju commented 2026-02-18 23:27:37 +01:00

Author

Member

Copy link

but other than that I guess its finished

but other than that I guess its finished

leuckeju commented 2026-02-18 23:27:38 +01:00

Author

Member

Copy link

resolved all threads

resolved all threads

leuckeju commented 2026-02-18 23:40:12 +01:00

Author

Member

Copy link

added 5 commits

• d2f04450 - Align uncompleted and completed todo items
• 12468a3a - Make calendar nav buttons full width on mobile
• 60033315 - Align slider in percentcompletefield to label
• 54f498e2 - Hide clock icon for datetime fields
• 1881cdae - Differ input field icon between datetime and date

Compare with previous version

added 5 commits <ul><li>d2f04450 - Align uncompleted and completed todo items</li><li>12468a3a - Make calendar nav buttons full width on mobile</li><li>60033315 - Align slider in percentcompletefield to label</li><li>54f498e2 - Hide clock icon for datetime fields</li><li>1881cdae - Differ input field icon between datetime and date</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2158/diffs?diff_id=83596&start_sha=2851e9fcacae1c98c2a0a0c9ae440e4cc8cf2c6c)

leuckeju commented 2026-02-20 12:53:39 +01:00

Author

Member

Copy link

marked this merge request as ready

marked this merge request as **ready**

leuckeju commented 2026-02-20 12:53:45 +01:00

Author

Member

Copy link

requested review from @hansegucker

requested review from @hansegucker

nik commented 2026-02-20 16:30:27 +01:00

Owner

Copy link

added 75 commits

• 1881cdae...04714638 - 74 commits from branch master
• 50f11bb3 - Merge branch 'master' into 'mobile-improvements'

Compare with previous version

added 75 commits <ul><li>1881cdae...04714638 - 74 commits from branch <code>master</code></li><li>50f11bb3 - Merge branch &#39;master&#39; into &#39;mobile-improvements&#39;</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2158/diffs?diff_id=83620&start_sha=1881cdae8ff553cffb2bdab00d3a309d90738158)

nik merged commit d3fb7c0368 into master 2026-02-20 16:34:23 +01:00

nik commented 2026-02-20 16:34:31 +01:00

Owner

Copy link

mentioned in commit d3fb7c0368

mentioned in commit d3fb7c03685cf3fc26a4502d83e668f3c7ae7e2e

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

Security

  

TeX

Needs LaTeX template

  

auto-update

  

board

done

  

board

ready

  

board

todo

  

check

delete-eslint-rc-js

  

check

update-builddeps-package-json

  

check

update-eslint-rc-js

  

check

update-gitignore

  

check

update-merge-request-template

  

check

update-prettier-ignore

  

check

update-pyproject-toml

  

check

update-renovate-json

  

check

update-tox-ini

  

part

backend

Backend (view code, models, logic, etc.)

  

part

ci

  

part

docs

Issue or addition to the documentation

  

part

frontend

Frontend (templates, etc.)

  

part

i18n

Translation or internationalisation issue

  

part

non-technical

Non-technical (policy, maintenance, etc.) issue or discussion

  

part

packaging

Issue concerning the packaging of AlekSIS and components

  

prio

1

Priority High

  

prio

2

Priority Medium

  

prio

3

Priority Low

  

release-mr-5.x

  

size

large

  

size

medium

  

size

small

  

source

customer

Requested by a customer or affiliated user

  

source

customer::fsmw

  

source

customer::fss

  

source

customer::teckids

  

source

downstream

Issue or merge request from other projects using AlekSIS or parts of it

  

type

breaking

  

type

bug

Bug

  

type

feature

New feature

  

type

refactoring

  

workflow

blocked

Issue is blocked by another issue

  

workflow

confirmed

The bug is reproducible and will be fixed

  

workflow

current-todo

  

workflow

discussing

Needs to be discussed before implementation

  

workflow

new-app

Requires new AlekSIS app, which has to be created

  

workflow

wontfix

The bug is not reproducible or the feature was decided against

  

source::aleksolotl

No labels

Security

TeX

auto-update

board

done

board

ready

board

todo

check

delete-eslint-rc-js

check

update-builddeps-package-json

check

update-eslint-rc-js

check

update-gitignore

check

update-merge-request-template

check

update-prettier-ignore

check

update-pyproject-toml

check

update-renovate-json

check

update-tox-ini

part

backend

part

ci

part

docs

part

frontend

part

i18n

part

non-technical

part

packaging

prio

1

prio

2

prio

3

release-mr-5.x

size

large

size

medium

size

small

source

customer

source

customer::fsmw

source

customer::fss

source

customer::teckids

source

downstream

type

breaking

type

bug

type

feature

type

refactoring

workflow

blocked

workflow

confirmed

workflow

current-todo

workflow

discussing

workflow

new-app

workflow

wontfix

source::aleksolotl

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

hansegucker

3 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

aleksis/AlekSIS-Core!3803

No description provided.