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

Theme added #54

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

Theme added #54

wants to merge 1 commit into from

Conversation

Akshat3144
Copy link

Add Theme Support to Calculator Application

Description
This pull request introduces a new feature that allows users to switch between different themes in the calculator application. The following changes have been made:

Theme Class: A new class has been created to encapsulate theme properties such as styles for combo boxes, labels, buttons, and background images.

Theme Instances: Multiple theme instances have been defined, each with unique styles and background images.

Apply Theme Function: A new function class to apply the selected theme to all UI elements.

Theme Selection: Actions for selecting different themes have been connected to the function.

Changes
Created theme.h in the Calculator folder to define the class.
Updated to include theme.h, add the function, define theme instances, and connect theme selection actions.
Added Appropriate Comments.

How to Test
Build and run the application.
Use the menu to select different themes.
Verify that the UI elements update according to the selected theme.

Additional Notes
More themes can be easily added by defining new instances of the class and connecting them to actions.
The current implementation includes two sample themes and Additional themes can be added as needed.
Checklist
Code compiles without errors.
All existing and new tests pass.
Documentation has been updated to reflect the changes.
Screenshots have been included for visual reference.

Thank you for reviewing this pull request. Please let me know if there are any questions or further improvements needed.

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.

1 participant