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

Support location random offset and long press to modify the history record name #162

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ZnDong
Copy link

@ZnDong ZnDong commented Sep 21, 2024

类型 / Type

  • 新特性 New feature
  • 问题修复 Bug fix
  • 编码风格优化 Code style optimization
  • 文档更新 documentation update
  • 依赖更新 Dependencies update
  • 性能优化 Performance optimization
  • 功能增加 Enhancement function
  • 其他 Other (about what?)

描述 / Description

  1. 由于APP自带的全球逆地理编码API配额有限,导致历史记录中极少包含名称,难以有效区分各个历史记录。通过新增修改历史记录名称的功能,用户可以将历史记录作为类似收藏列表来使用,方便管理与查找。
  2. 支持在使用历史记录中的位置时进行随机偏移,有效避免多次使用相同经纬度的位置。同时优化FragmentSettings.java中的部分代码,增强代码复用性。

相关 ISSUE / Related ISSUE

合并检测 / Merge Check

  • 文档是否更新 If document is updated or not
  • 变更记录是否记录 If changelog is updated or not

@ZnDong
Copy link
Author

ZnDong commented Sep 21, 2024

另外,目前APP的逻辑是在使用历史记录后会按照经纬度删除之前的历史记录,这有违历史记录的意义,还会因为多次的String和double互转导致精度丢失,进而导致之前的历史记录无法正确删除。

个人认为收藏列表比”类历史记录“更具有实用性,建议向收藏列表过渡:

  1. 长按可修改历史记录名称
  2. 不自动保存来自历史记录的位置
  3. 修复保存地点时名称可能是上次选择的历史记录的问题

如果以上建议开发者采纳,我将在本PR处理完成后继续提交以上PR。

相关Issue:
https://gitee.com/itexp/gogogo/issues/IAP1EF
https://gitee.com/itexp/gogogo/issues/IAIT2A

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.

1 participant