Skip to content
This repository was archived by the owner on Jun 18, 2026. It is now read-only.

ADD TEXTAREA - #118

Merged
sethhorsley merged 9 commits into
ruby-ui:mainfrom
pierry01:jean/add-textarea
Oct 1, 2024
Merged

ADD TEXTAREA#118
sethhorsley merged 9 commits into
ruby-ui:mainfrom
pierry01:jean/add-textarea

Conversation

@pierry01

@pierry01pierry01 commented Sep 23, 2024

Copy link
Copy Markdown
Contributor

ruby-ui/ruby_ui#139

image

The error will be fixed when RBUI merges the Textarea PR
image

Comment threadapp/views/docs/textarea_view.rb
@pierry01
pierry01 marked this pull request as ready for review September 23, 2024 23:23
@pierry01pierry01 mentioned this pull request Sep 23, 2024
Comment threadapp/views/docs/textarea_view.rb Outdated

def components
[
Docs::ComponentStruct.new(name: "Textarea", source: "lib/rbui/textarea/textarea.rb", built_using: :phlex)

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

source fixed
image

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.

Last update:
The views need to have this updated structure:
https://github.com/rbui-labs/rbui-web/pull/119/files#diff-91e4c03b6dc05c5672ab6eabbff23ed3bd234bc13e085b891e5ed52e91e17c16

This line will register all the code to the parser to extract the components being used and also show the files that the component has:
https://github.com/rbui-labs/rbui-web/blob/8431fdf1ff0feb45c0aa1643c6175f79d7944caa/app/views/docs/alert_dialog_view.rb#L30

@sethhorsleysethhorsley 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.

Looks great!

@sethhorsley
sethhorsley merged commit 107745d into ruby-ui:mainOct 1, 2024
djalmaaraujo pushed a commit to djalmaaraujo/web that referenced this pull request Apr 16, 2026
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@pierry01@sethhorsley