Skip to content

Refactor icon display logic for Windows and remove unnecessary text from menus#7

Merged
ITx-prash merged 3 commits into
mainfrom
break-and-build
Aug 1, 2025
Merged

Refactor icon display logic for Windows and remove unnecessary text from menus#7
ITx-prash merged 3 commits into
mainfrom
break-and-build

Conversation

@ITx-prash

Copy link
Copy Markdown
Owner

Summary

  • Removed ugly bracket icons ([BANK], [REG], etc.) from Windows interface
  • Deleted "Made with <3" text from all menus
  • Linux emoji icons preserved (🏦, 📝, 🔐)

Result

Cleaner, more professional Windows interface while maintaining Linux visual appeal.

Testing: ✅ Both platforms built and verified successfully

@ITx-prash ITx-prash requested a review from ignotus-8 August 1, 2025 19:36
@ITx-prash ITx-prash added the enhancement New feature or request label Aug 1, 2025
@ITx-prash ITx-prash merged commit 10054ff into main Aug 1, 2025
3 checks passed
@ITx-prash ITx-prash requested review from Copilot and removed request for ignotus-8 August 1, 2025 19:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the icon display logic to provide cleaner Windows interface while preserving Linux emoji functionality. The changes remove complex Windows emoji detection and simplify the user experience by eliminating promotional text.

  • Simplified Windows icon logic by removing emoji detection and bracket-style icons
  • Removed "Made with <3" promotional text from all menu displays
  • Fixed inconsistent exit icon usage in main function
Comments suppressed due to low confidence (1)

crow-bank-app/main.cpp:370

  • [nitpick] The success message format "[SUCCESS]" is inconsistent with the existing "[OK]" format that was replaced. Consider using "[OK]" to maintain consistency with the existing codebase pattern.
        return "[SUCCESS]";

Comment thread crow-bank-app/main.cpp
{ // Exit
clearScreen();
cout << "\n Thank you for exploring SecureBank!\n";
cout << "\n" << getIcon("exit") << " Thank you for exploring SecureBank!\n";

Copilot AI Aug 1, 2025

Copy link

Choose a reason for hiding this comment

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

This line will display an empty string on Windows since getIcon("exit") returns "" for Windows, resulting in inconsistent formatting. The original hardcoded symbol may have been intentional to ensure consistent display across platforms.

Suggested change
cout << "\n" << getIcon("exit") << " Thank you for exploring SecureBank!\n";
{
std::string exitIcon = getIcon("exit");
if (exitIcon.empty()) exitIcon = ">>";
cout << "\n" << exitIcon << " Thank you for exploring SecureBank!\n";
}

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants