keymanapp / keymanapp/keyman

feat(web): consolidate `setPos()` and `setRect()` as they have overlapping purposes

Open
#4,949 1 comment 0 reactions 0 assignees View on GitHub
feat m:osk web/ web/engine/
Dominant language
Pascal
Stars
534
Forks
143
Avg merge
2d 10h
Merged PRs (30d)
113

Description

**Describe the bug**

I would like to deprecate the `setPos()` function in favour of the `setRect()` function... It's confusing having both. Perhaps even extending `setRect()` to support an additional parameter which is `followActiveTextField: boolean`?

_Originally posted in https://github.com/keymanapp/help.keyman.com/issues/379_

---

**KeymanWeb:**
- OS: any
- Browser: any
- KeymanWeb Version: 14.0.271

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.