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

Fix execCommand link #245

Closed
wants to merge 1 commit into from
Closed

Fix execCommand link #245

wants to merge 1 commit into from

Conversation

saschanaz
Copy link
Member

@saschanaz saschanaz commented May 3, 2020

Since it's in ActiveDocuments directory.

Fixes #247

@marcoscaceres
Copy link
Member

Well, this it's complicated... it's a maintained document, but there is zero expectation of ever standardizing, right?

@saschanaz
Copy link
Member Author

@marcoscaceres It's already in that directory, only the link is broken.

@johanneswilm
Copy link
Contributor

Hey @gked - I suspect you accidentally moved the execCommand document into the ActiveDocuments folder. I assume you meant to put it into the Archive folder, right?

@johanneswilm johanneswilm added the Agenda+ Agenda item to be inserted in the Editing TF meeting queue label May 6, 2020
Copy link
Member

@marcoscaceres marcoscaceres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I should probably go back into archived. We can still maintain it when there is glaring inconsistencies/errors, but let's not give false hope that it's going anywhere.

@marcoscaceres
Copy link
Member

What might be useful is to figure out what we might be able to salvage and maybe push that into HTML? This may just be tiny little things, but every little thing we salvage is a win. There would have to be clear evidence of interoperability.

@johanneswilm
Copy link
Contributor

We decided on the call on Friday to do as you say and put it back into Archived. It does not prevent us from editing the file when need be.

@marcoscaceres
Copy link
Member

Sounds great @johanneswilm. Thanks for the update!

@saschanaz
Copy link
Member Author

saschanaz commented May 12, 2020

but let's not give false hope that it's going anywhere.

"Archived" also gives false information that those will never be maintained. It should really be "Inactive" or something. (And possibly also renaming ActiveDocuments to just Active, or even active.)

@saschanaz
Copy link
Member Author

This is fixed by #264.

@saschanaz saschanaz closed this Jul 13, 2020
@saschanaz saschanaz deleted the patch-2 branch July 13, 2020 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda+ Agenda item to be inserted in the Editing TF meeting queue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

execCommand archive link is broken
3 participants