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

Focus editor in replies and forwdared messagess #2479

Merged
merged 1 commit into from
Jan 15, 2020
Merged

Conversation

GretaD
Copy link
Contributor

@GretaD GretaD commented Jan 14, 2020

Ref #2373

Co-authored-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: GretaD <gretadoci@gmail.com>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst changed the title (wip) change reply focus Focus editor in replies and forwared messagess Jan 14, 2020
@ChristophWurst ChristophWurst changed the title Focus editor in replies and forwared messagess Focus editor in replies and forwdared messagess Jan 14, 2020
@@ -127,6 +131,9 @@ export default {
priority: 'highest',
}
)
if (this.focus) {
logger.debug('focusing TextEditor')
Copy link
Member

Choose a reason for hiding this comment

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

This should probably be removed? :)

Copy link
Member

Choose a reason for hiding this comment

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

I'm fine with leaving those as they aid debugging of bug reports.

@jancborchardt
Copy link
Member

Hmm, I can’t really confirm that it fixes anything? Also I still have myself in cc when replying to mails.

@ChristophWurst
Copy link
Member

Also I still have myself in cc when replying to mails.

#2454

@ChristophWurst
Copy link
Member

Hmm, I can’t really confirm that it fixes anything?

Have you built the app? What happens when you try to reply to a message? What is focused? Do you see the debug log about the focus?

@ChristophWurst ChristophWurst force-pushed the reply_issues branch 2 times, most recently from 4a21fb3 to 3d97d83 Compare January 15, 2020 16:00
@GretaD GretaD requested a review from jancborchardt January 15, 2020 16:37
@GretaD
Copy link
Contributor Author

GretaD commented Jan 15, 2020

@jancborchardt its working now, can you please recheck?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants