Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Refactoring dropdown properties #156

Closed

Description

@Marc-Andre-Rivet

Similar to what's been done for styles, the dropdown properties need to be improved to provide a more cohesive API.

Existing props / sub-props

  • column options
  • column_static_dropdown and column_conditional_dropdown
  • dropdown_properties

It's been established previously that:

  • conditional and default dropdowns are required/useful for certain scenarios
  • index based dropdowns are required/useful for certain scenarios

One approach would be to use the same "if" syntax as for styles (e.g. column_id="foo", row_index="123") to cover both use cases with a single property (or two properties, one for conditionals and one for base), getting rid of options and dropdown_properties and cleaning up both columns and getting rid of the untype-able prop.

This may lead to a "heavier" index based interface, which is or not acceptable. The goal here is both to implement a solution and to get the discussion started.

@cldougl @chriddyp

Activity

  1. alexcjohnson commented on Jun 27, 2019

    @alexcjohnson
    Collaborator

    Done as of #446

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions