-
-
Notifications
You must be signed in to change notification settings - Fork 431
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
Address macOS issue where right-clicking a username in the main chat #4523
Conversation
…(or game chat) areas would pop up a seemingly empty user profile. This is because the resize event is overridden and doesn't actually attempt to resize based on the size hint of the dialog. Now that we're explicit with the call, this resize should be forced and have comparable results to popping up user profile from the user list.
wait, why do we do something like assigning SIXTY as a constant, like what |
cockatrice/src/userinfobox.cpp
Outdated
const qint64 DAYS_IN_A_YEAR = 365; | ||
const quint64 SIXTY = 60; | ||
const quint64 HOURS_IN_A_DAY = 24; | ||
const quint64 DAYS_IN_A_YEAR = 365; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wait, why do we do something like assigning SIXTY as a constant, like what
I assume the intention was to name it MINUTES_IN_AN_HOUR
when looking at the other constant names.
I made a pr that will address this #4526 |
* use datetime for calculating account age make translating easier by using tr multiples automatically account for leap days
…(or game chat) areas would pop up a seemingly empty user profile. This is because the resize event is overridden and doesn't actually attempt to resize based on the size hint of the dialog. Now that we're explicit with the call, this resize should be forced and have comparable results to popping up user profile from the user list.
Related Ticket(s)
Before:
After: