Skip to content

Add Feres logo#59

Open
aymenhayderhtml-pixel wants to merge 11 commits into
Chapa-Et:mainfrom
aymenhayderhtml-pixel:feres-logo
Open

Add Feres logo#59
aymenhayderhtml-pixel wants to merge 11 commits into
Chapa-Et:mainfrom
aymenhayderhtml-pixel:feres-logo

Conversation

@aymenhayderhtml-pixel

@aymenhayderhtml-pixel aymenhayderhtml-pixel commented Apr 4, 2026

Copy link
Copy Markdown

Adds the Feres logo asset and updates logos.json.

Fixes #52.


Note

Low Risk
Low risk data-only change: adds a single new entry to logos.json with no behavioral or security impact.

Overview
Adds a new Feres record to logos.json (filename feres, URL https://feres.co/, category Taxi Hailing).

Reviewed by Cursor Bugbot for commit c976ff7. Bugbot is set up for automated code reviews on this repo. Configure here.

Added SVG file for the Feres logo with a green background and horse silhouette.
Add Feres PNG
Remove bad Feres SVG

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c976ff7. Configure here.

Comment thread logos.json Outdated
"url": "https://feres.co/",
"category": ["Taxi Hailing"],
"Hint": ""
},

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing SVG file causes broken Feres logo display

High Severity

The Feres entry sets "Hint": "" which implies a vector logo is available, but logos/feres/ only contains feres.png — there is no feres.svg. The application in imageLoad.js always uses the SVG path (logos/${filename}/${filename}.svg) as the <img> source, so the Feres logo will render as a broken image on the page. Every other entry with "Hint": "" has both a .png and .svg file. Either an SVG file needs to be added, or Hint needs to be set to "Vector Logo not available" (though even that won't fix the broken image since the code always loads SVG).

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit c976ff7. Configure here.

Comment thread logos.json Outdated
"url": "https://feres.co/",
"category": ["Taxi Hailing"],
"Hint": ""
},

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Inconsistent indentation in new JSON entry

Low Severity

The new Feres entry uses 2-space indentation for properties and no indentation for the closing brace, while every other entry in logos.json uses tab indentation (one tab for braces, two tabs for properties). This inconsistency breaks the formatting convention of the entire file.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit c976ff7. Configure here.

@cursor

cursor Bot commented Apr 4, 2026

Copy link
Copy Markdown

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@cursor

cursor Bot commented Apr 4, 2026

Copy link
Copy Markdown

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@cursor

cursor Bot commented Apr 4, 2026

Copy link
Copy Markdown

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@cursor

cursor Bot commented Apr 4, 2026

Copy link
Copy Markdown

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@cursor

cursor Bot commented Apr 4, 2026

Copy link
Copy Markdown

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

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.

Feres Logo

1 participant