Skip to content

feat(nextjs): suggested a new background for the nextjs background#4

Open
matthew-mcateer wants to merge 2 commits intomainfrom
modify_background_in_nextjs
Open

feat(nextjs): suggested a new background for the nextjs background#4
matthew-mcateer wants to merge 2 commits intomainfrom
modify_background_in_nextjs

Conversation

@matthew-mcateer
Copy link
Owner

Hey @Shubhamai & @ashenafee this is a pretty minor change.

Changed the default background image from that of the hackathon to something similar yet different.

Literally just uploaded the original background image to MidJourney, used the /describe endpoint, and then picked an image that looked similar yet was clearly different.

@matthew-mcateer matthew-mcateer requested a review from Shubhamai May 31, 2023 01:15
@Shubhamai
Copy link
Collaborator

Hi, the image looks good but it doesn't seem to fit well in the web app. We will either need an image with completely black borders, or we can apply this image to the entire background!

image

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

1 similar comment
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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