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

Allow for customizing the behavior of Right Clicking of usernames. #4622

Merged
merged 11 commits into from
Jul 1, 2023

Conversation

Mm2PL
Copy link
Collaborator

@Mm2PL Mm2PL commented May 10, 2023

Description

A follow up to #4424 and #4183

@Mm2PL Mm2PL changed the title Add username right-click customization Allow for customizing the behavior of Right Clicking of usernames. May 10, 2023
@Mm2PL Mm2PL requested a review from pajlada May 10, 2023 23:14
@Mm2PL Mm2PL added the Waiting for review PR bounced back to reviewer label May 10, 2023
@Mm2PL Mm2PL added this to the Post 2.4.4 milestone May 10, 2023
src/widgets/settingspages/GeneralPage.cpp Show resolved Hide resolved
src/widgets/settingspages/GeneralPage.cpp Outdated Show resolved Hide resolved
@Mm2PL Mm2PL requested a review from pajlada June 27, 2023 21:55
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

src/widgets/helper/ChannelView.cpp Outdated Show resolved Hide resolved
src/widgets/settingspages/GeneralPage.cpp Show resolved Hide resolved
@pajlada pajlada removed the Waiting for review PR bounced back to reviewer label Jul 1, 2023
@pajlada pajlada enabled auto-merge (squash) July 1, 2023 10:09
@pajlada pajlada merged commit 2f272b3 into master Jul 1, 2023
16 of 17 checks passed
@pajlada pajlada deleted the feature/lidl_hotkeys_for_username_right_click branch July 1, 2023 11:03
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

auto userMention = formatUserMention(
link.value, isFirstWord, commaMention);
insertText("@" + userMention + " ");
Qt::KeyboardModifier userSpecifiedModifier =
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warning: variable 'userSpecifiedModifier' of type 'Qt::KeyboardModifier' can be declared 'const' [misc-const-correctness]

Suggested change
Qt::KeyboardModifier userSpecifiedModifier =
Qt::KeyboardModifier const userSpecifiedModifier =

return;
}

Qt::KeyboardModifiers modifiers{userSpecifiedModifier};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warning: variable 'modifiers' of type 'Qt::KeyboardModifiers' (aka 'QFlagsQt::KeyboardModifier') can be declared 'const' [misc-const-correctness]

Suggested change
Qt::KeyboardModifiers modifiers{userSpecifiedModifier};
Qt::KeyboardModifiers const modifiers{userSpecifiedModifier};

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.

2 participants