Enable sharing URLs via email
Needs ReviewPublic

Authored by nicolasfella on Sat, Dec 1, 7:31 PM.

Details

Reviewers
apol
Summary

Share URLs via email. Add title to input data
Depends on D17285

Test Plan

Send page via Email from Falkon

Diff Detail

Repository
R495 Purpose Library
Branch
emailUrl
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 5589
Build 5607: arc lint + arc unit
nicolasfella created this revision.Sat, Dec 1, 7:31 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptSat, Dec 1, 7:31 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
nicolasfella requested review of this revision.Sat, Dec 1, 7:31 PM
apol added a comment.Tue, Dec 4, 2:40 AM

I'm not sure it makes a lot of sense to have the application include the title, isn't it better that the user writes it down on a config page or just the e-mail client?

In D17287#370993, @apol wrote:

I'm not sure it makes a lot of sense to have the application include the title, isn't it better that the user writes it down on a config page or just the e-mail client?

The user still can change/remove it in the email client