Skip to content

feat: split conditions and rules into separate tabs, refactor schema component #1121

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 48 commits into
base: master
Choose a base branch
from

Conversation

catosaurusrex2003
Copy link
Contributor

@catosaurusrex2003 catosaurusrex2003 commented Jan 11, 2025

The problem

The current schema UI assumes a deep understanding of the AsyncAPI specification, making it difficult for many users to interpret. Since schemas can be recursive and heavily nested with various properties, the complexity often becomes overwhelming and unintuitive.

The solution

This PR introduces a new UI structure that splits schema properties into two logical sections: Rules and Conditions, making the documentation easier to understand and navigate.

Reference: AsyncAPI Schema Object (v3.0.0)

The properties which come under the Rules section are:
format, pattern, constraints, contentEncoding, enum, default, const

The properties which come under the Conditions section are:
oneOf, anyOf, allOf, not, propertyNames, contains, if, then, else, dependentSchemas

All other properties remain in their original positions.

A tabbed interface now allows users to toggle between Rules and Conditions. This design reduces visual clutter and helps users digest smaller, relevant portions of the schema at a time.

Visual Comparison

Old schema UI 👇
image

New implementation - Rules tab 👇
image

New implementation - Conditions tab 👇
image

New implementation - Conditions tab and expand all 👇
image

Alternative approaches explored
We initially considered a sidebar-based layout as demonstrated here that separated Rules and Conditions spatially. However, it introduced several issues:.

  • Significant empty space when one section had more items than the other.

Without sidebar 👇 :
image

With sidebar 👇 : (you can notice how the empty spaces in red which is very unoptimal)
image

  • Reduced horizontal space for nested schemas
  • Poor responsiveness on smaller screens

After discussions with @AceTheCreator we opted for the tabbed layout, which offers better responsiveness, clarity, and overall user experience.

Other changes

  • The existing library/src/components/Schema.tsx has been refactored into smaller, modular components under a new library/src/components/Schema/ directory for better maintainability and readability.

  • Updated the way constraints are displayed: ranges like [0..1000] are now represented in a more intuitive format as 1 <= value <= 1000 for better readability.

  • Updated the start and build:styles:dev script to include all the available tailwind styles during development. This ensures that when you add a new Tailwind class (even one not yet used anywhere else in the project), you don’t need to rebuild the styles manually. The new styles are already available in default.min.css, so changes take effect instantly through hot reloading — speeding up the development workflow.

Source/Inspiration

#618
https://www.asmitbm.me/projects/asyncapi-docsux

This PR is part of https://github.com/orgs/asyncapi/discussions/1361#discussioncomment-10811342

@catosaurusrex2003 catosaurusrex2003 marked this pull request as ready for review February 25, 2025 11:47
Comment on lines 21 to 22
// const defaultSchema = specs.overcomplicatedStreetlights;
// const defaultSchema = specs.complexSchema;
Copy link
Member

Choose a reason for hiding this comment

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

Don't forget to remove this commented lines

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should I remove the newly added specs from playground/specs?
These specs were created purely for stress-testing the library and don’t reflect real-world use cases.

I can remove them to keep the repo clean or leave them for future reference in case someone needs it.

Copy link
Member

Choose a reason for hiding this comment

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

Yea, you can leave till the PR is finalized 👍🏾

@AceTheCreator
Copy link
Member

@catosaurusrex2003 one test is failing

@catosaurusrex2003
Copy link
Contributor Author

@catosaurusrex2003 one test is failing

@AceTheCreator yeah the test are failing due to the new changes and it requires for the test to be modified.
I was waiting for my changes to be finalized then I can modify the tests according to it.

@catosaurusrex2003 catosaurusrex2003 changed the title feat: refactor payload feat: introduce seperate conditions and rules tabs and refactor schema component Apr 10, 2025
@catosaurusrex2003 catosaurusrex2003 changed the title feat: introduce seperate conditions and rules tabs and refactor schema component feat: split conditions and rules into separate tabs, refactor schema component Apr 10, 2025
Copy link

Copy link

@catosaurusrex2003
Copy link
Contributor Author

@AceTheCreator
I updated the scripts and added description for it.

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