Improved navigation, added home button

[vedansh-5]

📌 Fixes

Fixes #139

---

📝 Summary of Changes

Added home button to go back to report section, users can also click the scum helper heading to go back to the reports section.

---

📸 Screenshots / Demo (if UI-related)

---

✅ Checklist

• I’ve tested my changes locally
• I’ve added tests (if applicable)
• I’ve updated documentation (if applicable)
• My code follows the project’s code style guidelines

---

👀 Reviewer Notes

Add any special notes for the reviewer here

Summary by Sourcery

Restore navigation to the report section by adding a Home button and making the Scrum Helper heading clickable.

New Features:

• Allow navigation back to the report section via a Home button or by clicking the Scrum Helper heading.

Bug Fixes:

• Fix issue Improve Navigation: Add Option to Return to Main Screen from Settings #139 to restore navigation back to the report section.

[sourcery-ai]

Reviewer's Guide

This PR improves navigation by adding a home button and making the header clickable to return to the report view, refactors event handler setup in popup.js, updates the HTML structure to include the new button and heading ID, and extends CSS with styles for the home button in both light and dark modes.

Class diagram for updated popup UI elements and event handlers

Loading

classDiagram
    class PopupUI {
      +Element homeButton
      +Element scrumHelperHeading
      +showReportView()
    }
    PopupUI : +addEventListener(homeButton, 'click', showReportView)
    PopupUI : +addEventListener(scrumHelperHeading, 'click', showReportView)
    PopupUI : +showReportView()

File-Level Changes

Change	Details	Files
Enhanced navigation controls in popup.js	Declared homeButton and scrumHelperHeading elements alongside existing toggles Removed duplicate settingsToggle/reportSection/settingsSection declarations Added click handlers on homeButton and scrumHelperHeading to invoke showReportView()	src/scripts/popup.js
Updated popup.html to include home button and clickable header	Assigned id="scrumHelperHeading" and cursor-pointer class to the main heading Inserted a homeButton before settingsToggle with FontAwesome icon Wrapped buttons in a flex container for layout alignment	src/popup.html
Added CSS rules for homeButton styling	Defined base styles (background, border, cursor, flex alignment) Added active state background for light and dark modes	src/index.css

Assessment against linked issues

Issue	Objective	Addressed	Explanation
#139	Provide a way to navigate back to the main screen from the settings screen.	✅
#139	Make the navigation intuitive for users.	✅

Possibly linked issues

• Improve Navigation: Add Option to Return to Main Screen from Settings #139: PR adds a home button and makes the heading clickable to allow users to return from settings to the main screen, addressing the issue.
• Improve Navigation: Add Option to Return to Main Screen from Settings #139: PR adds home button and makes heading clickable for navigation, addressing minor UI fixes.

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it. You can also reply to a
  review comment with @sourcery-ai issue to create an issue from it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time. You can also comment
  @sourcery-ai title on the pull request to (re-)generate the title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time exactly where you
  want it. You can also comment @sourcery-ai summary on the pull request to
  (re-)generate the summary at any time.
• Generate reviewer's guide: Comment @sourcery-ai guide on the pull
  request to (re-)generate the reviewer's guide at any time.
• Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
  pull request to resolve all Sourcery comments. Useful if you've already
  addressed all the comments and don't want to see them anymore.
• Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
  request to dismiss all existing Sourcery reviews. Especially useful if you
  want to start fresh with a new review - don't forget to comment
  @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[Copilot]

Pull Request Overview

This pull request improves navigation by adding a home button and making the "Scrum Helper" heading clickable to return to the reports section.

• Added new UI elements (home button and clickable heading) to trigger navigation back to the reports view.
• Removed duplicate variable declarations in popup.js to streamline the code.
• Updated CSS to include styles for the new home button.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
src/scripts/popup.js	Added event listeners and refactored duplicate declarations for navigation.
src/popup.html	Updated markup to include the clickable scrum helper heading and home button.
src/index.css	Added styling for the new home button to support UI consistency.

[sourcery-ai]

Hey @vedansh-5 - I've reviewed your changes - here's some feedback:

• Consider moving the inline font-size style on the home button into your CSS files to keep styling consistent and maintainable.
• Add an accessible label or aria-label to the home button icon so screen readers can identify its purpose.
• You might consolidate the click handlers for homeButton and scrumHelperHeading into a single function or event delegation pattern to reduce duplication.

Prompt for AI Agents

Please address the comments from this code review:
## Overall Comments
- Consider moving the inline font-size style on the home button into your CSS files to keep styling consistent and maintainable.
- Add an accessible label or aria-label to the home button icon so screen readers can identify its purpose.
- You might consolidate the click handlers for `homeButton` and `scrumHelperHeading` into a single function or event delegation pattern to reduce duplication.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[vedansh-5]

@hpdang @mariobehling Please take a look whenever you can, I'm open for any and all suggestions. Thank you.

[hpdang]

The implementation works well for me, I really like it. My only comment is that the two icons feel relative big compared to the overall UI.

[vedansh-5]

I have reduced the icon size.

[vedansh-5]

I have resolved the conflicts, the extension works fine with all the functionalities intact. Thank you!
@hpdang