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

Removed autocorrect/autocapitalise from filepaths on WebUI #9841

Closed
wants to merge 0 commits into from

Conversation

AceLewis
Copy link
Contributor

As stated in #9796 the autocorrect and autocapitalise should be removed from filepath inputs, this is a larger issue for mobile devices using the WebUI.

@@ -21,7 +21,7 @@ <h2 class="vcenter">QBT_TR(Download Torrents from their URLs or Magnet links)QBT
<fieldset class="settings" style="border: 0; text-align: left;">
<div class="formRow" style="margin-top: 6px;">
<label for="savepath" class="leftLabelLarge">QBT_TR(Save files to location:)QBT_TR[CONTEXT=HttpServer]</label>
<input type="text" id="savepath" name="savepath" style="width: 16em;" />
<input type="text" id="savepath" name="savepath" autocorrect="off" autocapitalize="none" style="width: 16em;" />
Copy link
Member

Choose a reason for hiding this comment

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

Can you move these attributes to the file's <form> element so that they apply to all inputs? Same for upload.html.

In the future we should probably put preferences_content.html inside a form element so that we can easily cover all inputs.

Copy link
Member

Choose a reason for hiding this comment

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

@Piccirello, apparently, the author abandoned it... What are we supposed to do?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Piccirello, apparently, the author abandoned it... What are we supposed to do?

I have been busy maybe this weekend I will do the changes, I am happy for anyone else to also implement them.

@Piccirello Piccirello added the WebUI WebUI-related issues/changes label Nov 12, 2018
@glassez
Copy link
Member

glassez commented Jan 25, 2019

This PR seems to be forgotten...
@Piccirello, is it really important? Don't you want to finish it? Otherwise we close it.

@AceLewis
Copy link
Contributor Author

This PR seems to be forgotten...
@Piccirello, is it really important? Don't you want to finish it? Otherwise we close it.

Yeah, I had forgotten about it, I have now done it. I did what @Piccirello wanted but it has been a long time and now there is a conflict.

@AceLewis
Copy link
Contributor Author

I don't know how to resolve the conflict easily. My best option would be to remove the commits to "preferences_content.html" and then open a new pull request for that file.

@glassez
Copy link
Member

glassez commented Jan 25, 2019

@AceLewis, you should squash all your commits into one and then rebase your branch on top of current qBittorrent master (and resolve conflicts during rebase).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebUI WebUI-related issues/changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants