Skip to content

adding unix permissions columnn#1

Open
gavasc wants to merge 1 commit intoaristotelesbr:mainfrom
gavasc:feature/list-perms
Open

adding unix permissions columnn#1
gavasc wants to merge 1 commit intoaristotelesbr:mainfrom
gavasc:feature/list-perms

Conversation

@gavasc
Copy link
Copy Markdown

@gavasc gavasc commented Feb 20, 2025

No description provided.

@aristotelesbr
Copy link
Copy Markdown
Owner

Hey @gavasc tks for contributing the permissions display feature! It's a great addition, but I have a few concerns about including it in the default layout:

Cross-platform compatibility: Unix permissions don't translate well to Windows systems and might not work correctly there.
User preference: Not everyone needs to see permissions by default.

I suggest we keep this functionality as:

Available in AVAILABLE_COLUMNS (as already implemented)
Not included in DEFAULT_COLUMNS. Accessible through a more flexible configuration system

We could implement:

  • A .lsdrc config file where users can set their column preferences
# .lsdrc - yml or toml

columns:
  - index
  - name
  - type
  - size
  - permissions # customize layout
  - modified    # other options

style:
  border: unicode
  rounded_corners: true
  
sort:
  by: name
  reverse: false

format:
  date: relative    # relative, iso, etc.
  size: human       # human, bytes, etc.

colors:
  name: cyan
  size: green
  modified: magenta
  • Environment variables like LSD_COLUMNS for temporary customization
  • An extensible system where users can add/remove columns as needed

This would make the tool more flexible without changing the default experience. What do you think?

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.

2 participants