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 Basic Footer Links #17

Open
Ansh5748 opened this issue Feb 21, 2025 · 1 comment · May be fixed by #18
Open

Add Basic Footer Links #17

Ansh5748 opened this issue Feb 21, 2025 · 1 comment · May be fixed by #18
Assignees

Comments

@Ansh5748
Copy link
Contributor

What happened:
Footer only shows copyright information

What did you expect to happen:
Additional links for GitHub, documentation, and version info

Error messages and logs:
N/A - Missing footer elements

@Ansh5748
Copy link
Contributor Author

Implementation Plan

Phase 1: Footer Enhancement

  • Add GitHub repository link
  • Add documentation link
  • Add version display
  • Maintain copyright notice

Phase 2: Styling

  • Implement flex layout for footer elements
  • Ensure consistent spacing
  • Add hover effects on links
  • Maintain responsive design

Code Changes

Footer Component

  • Update footer HTML structure
  • Add external links
  • Implement version display
  • Add target="_blank" for external links

Styling Updates

  • Add flex container styles
  • Implement link hover effects
  • Ensure mobile responsiveness
  • Maintain existing theme

Modified Files

  • flask_logging_server/templates/landing.html: Update footer structure
  • flask_logging_server/static/styles.css: Add footer styling

Test Strategy

Functionality Testing

  • Verify all links work correctly
  • Check external links open in new tabs
  • Validate copyright display
  • Test version number visibility

Responsive Testing

  • Mobile view alignment
  • Tablet layout verification
  • Desktop display checks

Ready to enhance the footer! 🚀

/cc @gtheodoridis @m-vasiloma Please assign this issue to me.

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 a pull request may close this issue.

1 participant