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

Update image credits for people.png #660

Closed
wants to merge 4 commits into from

Conversation

leewyatt
Copy link
Collaborator

@leewyatt leewyatt commented Sep 3, 2024

Update people image

Changed the author and source URLs for people.png to reflect the correct information. This ensures proper attribution and licensing compliance for the image used.
Simplified the conditional statements in the login function. Improved code readability and maintainability by splitting complex conditions into smaller, more descriptive variables.
…cement-people-image

# Conflicts:
#	mobile/src/main/resources/com/dlsc/jfxcentral2/mobile/components/people.png
@dlemmermann
Copy link
Collaborator

I completely replaced the graphics with new ones generated by AI.

@dlemmermann dlemmermann closed this Sep 4, 2024
@leewyatt
Copy link
Collaborator Author

leewyatt commented Sep 4, 2024

I completely replaced the graphics with new ones generated by AI.

I just tested it, and it looks great! The unified style will make the design look much more consistent and professional.

@dlemmermann dlemmermann deleted the enhancement-people-image branch September 5, 2024 14:33
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