MyFinances icon indicating copy to clipboard operation
MyFinances copied to clipboard

Issue #304 fix

Open jasonfy2k opened this issue 1 year ago • 2 comments

Description

Reworked the dropdown boxes to remove the if statements and instead work dynamically, opening upwards or downwards depending on if they would open past the viewport.

Checklist

  • [x] Ran the Black Formatter and djLint-er on any new code (checks will fail without)
  • [x] Made any changes or additions to the documentation where required
  • [x] Changes generate no new warnings/errors
  • [x] New and existing unit tests pass locally with my changes

What type of PR is this?

  • 🐛 Bug Fix

Added/updated tests?

  • 🙅 no, because they aren't needed

Related PRs, Issues etc

  • Related Issue #304
  • Closes #304

jasonfy2k avatar Dec 13 '24 00:12 jasonfy2k

Great thank you @jasonfy2k - I'll check this out later today

TreyWW avatar Dec 13 '24 07:12 TreyWW

Apologies for the delay on this, will take a look at once i've closed my PR due to merge conflicts. Bad management on my part 🫠

TreyWW avatar Jan 20 '25 10:01 TreyWW