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

Adapt style.css for mobile devices #1425

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

fnkbsi
Copy link
Contributor

@fnkbsi fnkbsi commented Mar 28, 2024

For a better administration of Steve from a mobile device the style.css could use some changes.

Add differentiation between mobile devices and desktop in style.css. Add meta tag with viewport attribute to the html code. 00-charging-profile.jsp: At the removeRow button in periodsTable changed class=removeRow to id=removeRow. Add to both buttons in periodsTable the attribute class="btnScheduleRow" (use this class in css). In the snippet schedulePeriodsTable.js change the selector from class (.removeRow) to id (#removeRow)

Add meta tag with viewport attribute to the html code.
00-charging-profile.jsp: At the removeRow button in periodsTable changed class=removeRow to id=removeRow. Add to both buttons in periodsTable the attribute class="btnScheduleRow" (use this class in css). In the snippet schedulePeriodsTable.js change the selector from class (.removeRow) to id (#removeRow)
@goekay goekay marked this pull request as ready for review June 23, 2024 07:53
@goekay
Copy link
Member

goekay commented Jun 23, 2024

hey @fnkbsi are these changes finished and stable?

@fnkbsi fnkbsi marked this pull request as draft June 23, 2024 17:30
@fnkbsi
Copy link
Contributor Author

fnkbsi commented Jun 23, 2024

@goekay: The style looks only good on some smartphones. I can't recommend to merge this PR in master. Currently I can't work on it, so should I delete the PR or keep it as inspiration for others?

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