Skip to content

feat(ratings): enable rating for installed add-ons - #2288

Open
Tweekazoid wants to merge 2 commits into
mainfrom
addon_rate
Open

feat(ratings): enable rating for installed add-ons#2288
Tweekazoid wants to merge 2 commits into
mainfrom
addon_rate

Conversation

@Tweekazoid

@Tweekazoid Tweekazoid commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

and improve UI feedback

This modifies what is shown in hover over thumbnail
and what is shown in detail view

@Tweekazoid
Tweekazoid requested a review from agajdosi August 28, 2026 23:35
@Tweekazoid
Tweekazoid requested a review from vilemduha August 28, 2026 23:44

@agajdosi agajdosi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ratings in thumbnail hover for addons does not work in my case, it is empty -. On asset popup it is shown correctly. (https://www.blendkit.com/asset-gallery-detail/63177030-cdb4-45ae-8bec-76a3ee48d151/)

Jinak najs :). vypada to hezky a dava smysl pro uzivatele IMHO.

Comment thread ratings.py

# Complexity ("working hours") rating is hidden for add-ons - it confuses
# regular users and doesn't map well to installable tools.
if self.asset_type == "addon":

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

Comment thread ratings.py

# Add-ons can only be rated once they are installed.
if self.asset_type == "addon":
from . import download

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tohle bych presunul na zacatek filu.
https://peps.python.org/pep-0008/#imports

Comment thread ui_panels.py
row.label(text=" ")

tooltip_extension = f".\n\nRatings results are shown for assets with more than {show_rating_threshold} ratings"
# Add-ons show only the quality star - the score/complexity confuse

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

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