Skip to content

Conversation

@csc-felipe
Copy link
Contributor

@csc-felipe csc-felipe commented Mar 8, 2023

Description

Many of buefy components used could already be changed to use csc-ui components.

Changes Made

  • Replace buefy b-input with c-text-field from csc-ui
  • Replace buefy b-button with c-button from csc-ui
  • Replace buefy b-select with c-select from csc-ui
  • Replace buefy b-loading with c-loader and remove unused b-loading
  • Replace buefy b-table with c-data-table from csc-ui

Testing

  • Tests do not apply
  • Manually test changed functionality. Changed components
    • CopyFolderModal.vue - input fields
    • CreateFolderModal.vue - input fields
    • ObjectTable.vue - search box
    • SelectPage.vue - text input, dropdown, buttons
    • Tokens.vue - text input, dropdown, buttons, data table

Mentions

As pointed out by @blankdots, buefy text fields look quite different from csc-ui, as csc-ui text fields look like the ones in material design from Google.

cc @hannyle, @ainoc

  • csc-ui
    image
  • buefy
    image

@csc-felipe csc-felipe added enhancement New feature or request debt Technical debt labels Mar 8, 2023
@csc-felipe csc-felipe self-assigned this Mar 8, 2023
@csc-felipe csc-felipe force-pushed the feature/replace-buefy-components branch from 5e3b1f7 to ecc7f40 Compare March 8, 2023 10:15
@csc-felipe
Copy link
Contributor Author

rebased

@csc-felipe csc-felipe force-pushed the feature/replace-buefy-components branch from ecc7f40 to b5be0ad Compare March 8, 2023 11:06
@csc-felipe csc-felipe force-pushed the feature/replace-buefy-components branch from b5be0ad to 6559bc4 Compare March 8, 2023 12:12
@csc-felipe csc-felipe requested a review from hannyle March 8, 2023 12:13
@csc-felipe
Copy link
Contributor Author

resolved comments

@csc-felipe csc-felipe requested a review from blankdots March 8, 2023 12:13
blankdots
blankdots previously approved these changes Mar 8, 2023
@csc-felipe
Copy link
Contributor Author

Removed custom-class from c-text-field

@csc-felipe csc-felipe requested a review from hannyle March 8, 2023 13:01
@csc-felipe csc-felipe merged commit 108af3c into devel Mar 9, 2023
@csc-felipe csc-felipe deleted the feature/replace-buefy-components branch March 9, 2023 08:12
blankdots pushed a commit that referenced this pull request Jun 26, 2023
Sharing to the same container twice did not show error toast

Closes #1016

See merge request sds-dev/sd-connect/swift-browser-ui!78
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debt Technical debt enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants