Skip to content

Fix: wrong cursor location, added battery optimization disabling screen and styling to the slash command in input field #120

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jul 19, 2025
Merged

Fix: wrong cursor location, added battery optimization disabling screen and styling to the slash command in input field #120

merged 3 commits into from
Jul 19, 2025

Conversation

anonfaded
Copy link
Contributor

@anonfaded anonfaded commented Jul 14, 2025 •
edited
Loading

Description

  • Now the cursor will auto move ahead of the slash command when we select a slash command from the list.
  • Added some styling to the command so it looks better and doesn't mix with other text in input field.
  • Added battery optimization permission and management system to ensure app runs reliably in background.

Screenshots

image image image

Closes: #119

Checklist

anonfaded added 2 commits July 14, 2025 14:23
@anonfaded
feat(battery): add battery optimization management for background rel…
fe57c56 
…iability

Add battery optimization permission and management system to ensure app runs reliably in background:
- Add REQUEST_IGNORE_BATTERY_OPTIMIZATIONS permission
- Create BatteryOptimizationManager to handle battery optimization checks and requests
- Add battery optimization UI screens and flow in onboarding
- Integrate battery optimization checks into MainActivity flow
@anonfaded
fix(chat): cursor location and enhance message input with slash comma…
21dc1a5 
…nd styling

- Change message input from String to TextFieldValue for better cursor control
- Add SlashCommandVisualTransformation to style slash commands with colored backgrounds
- Maintain cursor position when selecting command suggestions
@anonfaded anonfaded changed the title Fix: wrong cursor location and add styling to the slash command in input field Fix: wrong cursor location, added battery optimization disabling screen and styling to the slash command in input field Jul 14, 2025
@anonfaded anonfaded mentioned this pull request Jul 14, 2025
Open
@moehamade moehamade mentioned this pull request Jul 17, 2025
Closed
4 tasks
@callebtc
Copy link
Collaborator

hey thank you, this is a great PR. I've tested it and would love to merge it. we've just merged #129 that refactored the onboarding state. would you mind bringing your PR up to speed with it?

@anonfaded
Copy link
Contributor Author

hey thank you, this is a great PR. I've tested it and would love to merge it. we've just merged #129 that refactored the onboarding state. would you mind bringing your PR up to speed with it?

Yeah no problem, i will update my PR today

@anonfaded
Copy link
Contributor Author

hi @callebtc,
i have resolved the conflicts and also replaced the battery related hardcoded texts with strings.

@callebtc callebtc merged commit 04a774a into permissionlesstech:main Jul 19, 2025
1 check passed
@callebtc
Copy link
Collaborator

Very nice PR, hope we'll see more of your contributions!

@anonfaded
Copy link
Contributor Author

Very nice PR, hope we'll see more of your contributions!

Thank you! :)

@hector6872
Copy link
Contributor

@anonfaded we should anchor the buttons at the bottom and make the permission rationale scrollable to support smaller screens

https://x.com/franz_fk19/status/1946542465843920965?t=v04pMbXolxcHoREj1L-7ug&s=19

@anonfaded
Copy link
Contributor Author

@anonfaded we should anchor the buttons at the bottom and make the permission rationale scrollable to support smaller screens

https://x.com/franz_fk19/status/1946542465843920965?t=v04pMbXolxcHoREj1L-7ug&s=19

Yeah i already did that and opened the PR:) you can check the latest PR, it has screenshots

@hector6872
Copy link
Contributor

About 50 minutes ago. That was fast! 💪

Sorry for not double-checking before sending.

@anonfaded
Copy link
Contributor Author

About 50 minutes ago. That was fast! 💪

Sorry for not double-checking before sending.

yeah i faced the issue myself after trying the latest version so i instantly fixed that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] Cursor is not placed ahead of slash command
3 participants
@anonfaded @callebtc @hector6872