Skip to content
This repository has been archived by the owner on Feb 8, 2024. It is now read-only.

Fix grammar for non/interactive session event log #124

Merged
merged 4 commits into from
Jul 15, 2020
Merged

Conversation

kimlisa
Copy link
Contributor

@kimlisa kimlisa commented Jul 15, 2020

resolves gravitational/teleport#4051
fixes gravitational/teleport#4035

Description

  • Make vertical/horizontal scrollbar appear on content overflow (small window size)
  • Fix grammar for non/interactive session event log

Screenshot

grammar correction:
Screenshot from 2020-07-15 12-00-06
Screenshot from 2020-07-15 12-03-20
Screenshot from 2020-07-15 12-20-01

vertical/horizontal scrollbar:
screenshot_5

@kimlisa kimlisa requested review from alex-kovoy and webvictim July 15, 2020 19:20
@kimlisa kimlisa changed the title Lisa/grammar fix Fix grammar for non/interactive session event log Jul 15, 2020
@@ -397,13 +397,13 @@ export const formatters: Formatters = {
event.server_hostname || event.server_addr || event.server_id;

if (!event.interactive) {
return `User [${user}] has ended non-interactive session [${event.sid}] on node [${node}] `;
return `User [${user}] has ended an non-interactive session [${event.sid}] on node [${node}] `;
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be a non-interactive session rather than an non-interactive session

Copy link
Contributor Author

Choose a reason for hiding this comment

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

revised: dc32916

Copy link
Contributor

@webvictim webvictim left a comment

Choose a reason for hiding this comment

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

As per comment

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

Successfully merging this pull request may close these issues.

Grammar used in event log is weird Cluster List View is missing vertical scrollbars (4.3)
3 participants