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

disable editing of writing systems in LF (all edits of writing systems should happen in FLEx) #1553

Closed
megahirt opened this issue Oct 25, 2022 · 3 comments · Fixed by #1562
Closed

Comments

@megahirt
Copy link
Collaborator

No description provided.

@laineyhm
Copy link
Collaborator

Would this mean getting rid of the + button here:

image

And the input systems tab here:

image

?

@laineyhm
Copy link
Collaborator

-get rid of the three dots menu
-keep RTL setting, abbreviation, and font name
-other settings should be displayed but not editable
-get rid of new button in input systems tab and under input systems in configuration

@rmunn
Copy link
Collaborator

rmunn commented Oct 26, 2022

For #1552, I proposed redesigning the option lists to be keyed by GUID, which would allow us to restore the edit feature eventually. But for this one, I agree with removing the edit feature. I don't see a way to allow writing system edits in LF without massive Send/Receive conflicts. There might be a way involving a massive rearchitecture, but I don't have ANY good ideas. (Only some bad ones that are so bad they're not worth considering). So I agree with removing the ability for LF to edit writing systems, and not restoring that feature in the future (unlike option lists, which I think we will be able to edit in the future once we rearchitect them).

Repository owner moved this from To Do to Awaiting Testing (QA) in Language Forge Classic Oct 26, 2022
@longrunningprocess longrunningprocess moved this from Awaiting Testing (QA) to Awaiting Deployment in Language Forge Classic Oct 26, 2022
@longrunningprocess longrunningprocess moved this from Awaiting Deployment to Done in Language Forge Classic Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants