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

Improve the login dialog #335

Merged
merged 17 commits into from
Dec 13, 2023
Merged

Improve the login dialog #335

merged 17 commits into from
Dec 13, 2023

Conversation

loathingKernel
Copy link
Contributor

@loathingKernel loathingKernel commented Dec 12, 2023

Changes

  • Make the browser login from more readable (hopefully) and add privacy notice
  • Fix more issues with legendary's newly introduced file locking mechanism and how Rare uses legendary.

Internal changes

  • Stop login dialog from resizing when changing widgets
  • Remove not used hint_label from IndicatorLineEdit
  • Properly find the row to insert fields into in widgets using QFormLayout
  • Implement a slightly better concurrent and cachingQtRequests module
  • Disable some broken store features by hiding their widgets (search related)
  • Rename shop module to store to start incorporating cosmetic changes from https://github.com/loathingKernel/Rare/tree/shop_refactor

…ng widgets

The forms were updated to be more informative and easier to read.

NOTE: The reason the form is bigger than the contents is because the
QLabels have word-wrapping enabled and as such they substitute width
for height (`hasHeightforWidth`)
It looks like it is not used by anything any more.
* Properly find row to insert sid_edit into
* Use lazy evaluation for logging
* Clean the login forms by using separate layouts for the interactive fields
It is still junky but less so. It allows us to use QNetworkRequest's
caching mechanism instead of doing so on our own per-case.
Since Python 3.11, `FileLock` is thread-local by default, which causes
numerous issues with Rare because of numerous operations running in
`QThreads` and `QRunnables`. To work around it, add a monkey LGDLFS class
that uses a non-thread-local instance of `FileLock`. Since the monkey class
exists, implement a `unlock_installed` method for code clarity

* Add decorate `LegendaryCore.egl_sync` with `unlock_installed`
* Log that Rare's monkeys are in use
* Add function signature protocols based on `typing.Protocol`
Revert this once we move to python `>=3.10`
@loathingKernel loathingKernel merged commit d36582b into RareDevs:main Dec 13, 2023
1 check passed
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