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

Add div and class to confidential text for guests #5554

Merged
merged 1 commit into from Sep 15, 2017
Merged

Conversation

@Ruud68
Copy link
Contributor

@Ruud68 Ruud68 commented Sep 13, 2017

Currently the text for guests that the message contains confidential information is formatted as regular text in the message. This makes it confusing for quests as they do not understand what this means / why the author typed that.
With this patch the confidential text for guests is placed in a div with class kmsgtext-confidentialguests and by doing that the text can be styled via css to make it look different then the rest of the message.

Pull Request for Issue # .

Summary of Changes

Testing Instructions

Currently the text for guests that the message contains confidential information is formatted as regular text in the message. This makes it confusing for quests as they do not understand what this means / why the author typed that.
With this patch the confidential text for guests is placed in a div with class kmsgtext-confidentialguests and by doing that the text can be styled via css to make it look different then the rest of the message.
@Ruud68
Copy link
Contributor Author

@Ruud68 Ruud68 commented Sep 13, 2017

@810 810 added this to the 5.0.12 milestone Sep 15, 2017
@810 810 merged commit c673d56 into Kunena:K5.0 Sep 15, 2017
1 check failed
1 check failed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
810 added a commit that referenced this pull request Sep 15, 2017
@Ruud68 Ruud68 deleted the Ruud68:patch-2 branch Feb 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants