Skip to content
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

Allow users to control default variables (maybe via options) #77

Open
matt-dray opened this issue Jul 11, 2022 · 1 comment
Open

Allow users to control default variables (maybe via options) #77

matt-dray opened this issue Jul 11, 2022 · 1 comment
Labels
discuss Point for discussion enhancement New feature or request

Comments

@matt-dray
Copy link
Collaborator

Variables are hard-coded into scripts, like default column width, font size, etc. Could provide some sort of config file/allow the user the chance to set options().

The main user-facing functions should remain as simple as possible, given the package-design philosophy (i.e. arguments in new_a11ytable() should just be the components required to build an a11ytable).

@matt-dray matt-dray added enhancement New feature or request code labels Jul 11, 2022
@matt-dray matt-dray added this to the v0.2.0 milestone Jul 11, 2022
@matt-dray matt-dray removed the code label Dec 27, 2023
@matt-dray matt-dray changed the title Allow users to control default variables Allow users to control default variables (maybe via options) Dec 28, 2023
@matt-dray matt-dray added the discuss Point for discussion label Dec 28, 2023
@matt-dray
Copy link
Collaborator Author

One option might be cell width. The package currently uses a set width and doubles it if a certain character count is met by the table contents. Might be good to offer users the chance to change these 'default' and 'extended' values, and maybe allow for auto-width (bearing in mind that there will need to be a manual calculation for column A, since auto-width would adjust to the length of the longest of the arbitrary pre-table metadata rows or sheet title in most instances, rather than the content of the first column in the table.

@matt-dray matt-dray removed this from the v0.3.1 milestone Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Point for discussion enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant