Skip to content

Conversation

@bobur-khay
Copy link
Contributor

@bobur-khay bobur-khay commented Feb 3, 2026

I was suggested to use the filtering panel from JSON Schema but I came to the conclusion that using that amount of space for the only 3 filters is not worth it. The current implementation is the most efficient space usage imo.

@netlify
Copy link

netlify bot commented Feb 3, 2026

Deploy Preview for wot-marketing ready!

Name Link
🔨 Latest commit 31f61eb
🔍 Latest deploy log https://app.netlify.com/projects/wot-marketing/deploys/698358972ecfcc000892a3e8
😎 Deploy Preview https://deploy-preview-542--wot-marketing.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bobur-khay bobur-khay force-pushed the feature/developer-tools-filtering branch from 3375164 to 13dec14 Compare February 3, 2026 13:21
@danielpeintner
Copy link
Contributor

May I ask some questions w.r.t. images that are used for programming languages

  • Do we need images or is it sufficient to use text
  • I would never use a path with spaces "docs/images/programming languages/" --> better use hyphen
  • The netlify preview of images does not work (see https://deploy-preview-542--wot-marketing.netlify.app//developers/). I assume it works locally
  • we are mixing png and svg (might not be a problem)

@bobur-khay
Copy link
Contributor Author

May I ask some questions w.r.t. images that are used for programming languages

  • Do we need images or is it sufficient to use text
  • I would never use a path with spaces "docs/images/programming languages/" --> better use hyphen
  • The netlify preview of images does not work (see https://deploy-preview-542--wot-marketing.netlify.app//developers/). I assume it works locally
  • we are mixing png and svg (might not be a problem)
  • Good point on images. Substituted with text. Looks better.
  • Also a good point on path names
  • @egekorkan said it's a known issue with netlify deployments

@danielpeintner
Copy link
Contributor

FYI: You still link to images.. it looks like text on https://deploy-preview-542--wot-marketing.netlify.app//developers/ because the browser shows the alternate text if it cannot find the image which is now the case...

@bobur-khay
Copy link
Contributor Author

FYI: You still link to images.. it looks like text on https://deploy-preview-542--wot-marketing.netlify.app//developers/ because the browser shows the alternate text if it cannot find the image which is now the case...

That is still the old version, there is a deployment issue right now that I am working on.

@bobur-khay bobur-khay marked this pull request as draft February 4, 2026 10:22
@bobur-khay bobur-khay marked this pull request as ready for review February 4, 2026 10:28
@bobur-khay
Copy link
Contributor Author

Build fixed, could you have a look again please @danielpeintner

@egekorkan
Copy link
Contributor

egekorkan commented Feb 4, 2026

Sorry I got to review only now. Some remarks:

  • Filtering based on the platform would be good. E.g. Playground and EdiTDor are mainly browser facing tooling. Can be a separate PR.
  • Documenting the criteria for obsolete and making this visible on the page.
  • tmtd and wot-dtdl are not Other but more on the Thing Description category for me.

Otherwise, I really like the clean design and using yaml file to manage it all.

@bobur-khay
Copy link
Contributor Author

bobur-khay commented Feb 4, 2026

Sorry I got to review only now. Some remarks:

  • Filtering based on the platform would be good. E.g. Playground and EdiTDor are mainly browser facing tooling. Can be a separate PR.
  • Indicating the
  • Documenting the criteria for obsolete and making this visible on the page.
  • tmtd and wot-dtdl are not Other but more on the Thing Description category for me.

Otherwise, I really like the clean design and using yaml file to manage it all.

@egekorkan
Copy link
Contributor

The sentence of the second point is cut off

Sorry, removed it now :)

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.

3 participants