-
Notifications
You must be signed in to change notification settings - Fork 7
Main page logo changed by screenshot #10
Main page logo changed by screenshot #10
Conversation
index.html
Outdated
@@ -143,8 +143,11 @@ <h1> | |||
<div class="image-container"> | |||
<img | |||
class="img-fluid" | |||
src="img/ic_launcher-web.png" | |||
src="img/shot_1.webp" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might look nicer if the screenshot were showing an in-app view IMO. Other than that, it is approvable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, a screenshot of maybe the home page when the app is opened would be preferred over the web UI of trebleshot. This screenshot seems to be a better candidate.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I am trying to do changes according to you.
|
I have done changes, please review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good. Thanks for the help.
BTW, there is an unnecessary file named "git" in the root folder, but it is not a dealbreaker :)
We can discuss this by opening another issue.
Yes, I mentioned it in #7. |
I have not added this git file and don't even noticed. You may delete this :) |
I have replaced Logo in the main page by screenshot. Check if it is looking better or not.