Skip to content

Conversation

Cokser
Copy link

@Cokser Cokser commented Mar 3, 2025

Changes:

  • Added new property config, that has array actions
  • Array actions is related to all toolbar and context-menu actions
  • Each object in actions is customizable for the user and looks like this:
{
  key: string;
  title?: string;
  onClick?: Function;
  showToolbar?: boolean;
  showMenu?: boolean;
  multiple?: boolean;
  applyTo?: string[] | undefined; // TBD
  icon?: ReactNode;
  hidden?: boolean;
};
  • In FileManager Component I added default actions with properties, that user can overwrite it by key from actions array
  • Updated README.md for docs, // old actions to be removed.

Current changes applies only to selected items (Files or folders).
I'll add separate array for general actions next to actions, in the config property.

This change should cover some open issue like:

@Saifullah-dev Needs to be reviewed and tested.

@Saifullah-dev
Copy link
Owner

Saifullah-dev commented Mar 4, 2025

@Cokser Thank you for the contribution, looks great. I'll review and merge it in a day or two.

@Cokser
Copy link
Author

Cokser commented Mar 5, 2025

@Saifullah-dev let me know if there's any changes or fixes needed

@Saifullah-dev
Copy link
Owner

@Cokser Will provide an update on it today.

@Cokser Cokser mentioned this pull request Mar 11, 2025
@erhany
Copy link

erhany commented Mar 13, 2025

@Saifullah-dev

@Saifullah-dev
Copy link
Owner

Saifullah-dev commented Mar 13, 2025

@Cokser @erhany Extremely sorry, I'm quite busy nowadays with work. I'll add the translation and customize context menu features over the weekend.

@erhany
Copy link

erhany commented Mar 19, 2025

@Cokser @erhany Extremely sorry, I'm quite busy nowadays with work. I'll add the translation and customize context menu features over the weekend.

@Saifullah-dev :)

@erhany
Copy link

erhany commented Mar 26, 2025

@Cokser @erhany Extremely sorry, I'm quite busy nowadays with work. I'll add the translation and customize context menu features over the weekend.

@Saifullah-dev

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