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

Fix: Improved Simplified Logs table headers formatting #7

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Raakshass
Copy link

This PR addresses the issue #6: Improve Table Header Formatting in Simplified Logs. Previously, the headers in the simplified logs table were concatenated without spaces, making them difficult to read.

What was fixed:
Problem: Table headers were concatenated into a single string (e.g., "IDIPPortVersionCommandTypeTermMatchesStatusLevelMessageTime").
Expected Outcome: Clear and readable table headers with proper spacing and formatting.
Changes Implemented:
Redesigned the table headers to ensure proper spacing and improved readability.
Implemented responsive design to ensure the table headers remain accessible on various screen sizes.
Code Changes:
Added New Files:
table_styles.css: Contains styles for the table headers.
table_formatter.js: Implements the logic for formatting the headers.
Modified Existing Files:
simplified_logs.html: Updated the structure of the table.
styles.css: Added table-related improvements for better styling.
Testing Strategy:
Verified that the header alignment is consistent across different screen sizes.
Ensured the table is responsive and behaves as expected.
Checked that the updated headers are readable and well-spaced.

@gtheodoridis
Copy link
Collaborator

I do not really get the fix on this, can you provide a screenshot of the change?

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