Update hotkey.lua: add isActive

[muescha]

add isActive

See:

• Feature Request: I would like to know if a modal is in the state active #3607 Feature Request: I would like to know if a modal is in the state active

[github-actions]

This pull request does not contain a valid label. Please add one of the following labels: ['pr-fix', 'pr-change', 'pr-feature', 'pr-maintenance']

[muescha]

TODO:

• add docs if accepted
• should it be better as function like htm:isActive()?

[github-actions]

View Test Results

2 tests   2 ✔️  0s ⏱️
2 suites  0 💤
1 files    0 ❌

Results for commit d78aeef.

[muescha]

i can not see why the test are failing :(

[cmsj]

@muescha ignore the CI tests, they're horribly broken at the moment. I think this feature makes sense to add, so if you'd like to go ahead with it, I like your idea that an :isActive() method would be a better choice overall (so the precise implementation is hidden and can be changed later). With docs and that method, this can merge.

[muescha]

did you mean "hiding implementation" like this in 7e389a0

• removing from metatable
• renaming value to _isActive
• creating :isActive() method

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 27.63%. Comparing base (36c6495) to head (7e389a0).
Report is 817 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3608      +/-   ##
==========================================
+ Coverage   27.61%   27.63%   +0.01%     
==========================================
  Files         180      191      +11     
  Lines       36181    43465    +7284     
==========================================
+ Hits         9993    12012    +2019     
- Misses      26188    31453    +5265