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/a11y] Fix accessibility issues #1349

Merged
merged 31 commits into from
Jun 11, 2024
Merged
Show file tree
Hide file tree
Changes from 27 commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
0ba5b3e
- bump version to 12.3 and build to 291
felix-schwarz May 7, 2024
5a6cc66
- PrivacyInfo.xcprivacy: fix merge error from 04df106510cbab229ec11a6…
felix-schwarz May 13, 2024
4bbe61e
- INIFile: simple parse for .ini-file format files
felix-schwarz Apr 8, 2024
8b5b07e
- URLDisplayViewController: implement code to display and allow openi…
felix-schwarz Apr 11, 2024
4fe3d80
- Action: add new `.directOpen` location to allow actions to directly…
felix-schwarz Apr 16, 2024
ffcae3d
- regrouping shortcut actions into their own folder
felix-schwarz Apr 17, 2024
0a0e58c
- OpenShortcutFileAction: use correct (localized) string in alert button
felix-schwarz Apr 17, 2024
a16f6af
- update SDK to fix an issue arising from Available Offline policies …
felix-schwarz Apr 8, 2024
9d6e456
- Shortcuts:
felix-schwarz Apr 18, 2024
e234e8c
- OpenShortcutFileAction, ShortcutFileDisplayViewController, INIFile+…
felix-schwarz Apr 18, 2024
51692f2
- Fix: selecting the root folder of a space or the personal folder's …
felix-schwarz May 3, 2024
b0e448d
Configuration documentation updated
felix-schwarz May 3, 2024
233bdc5
Configuration documentation updated
felix-schwarz May 7, 2024
1862e4f
- PrivacyInfo.xcprivacy: remove extraneous space
felix-schwarz May 13, 2024
c0acb89
- swiftlint: disable large_tuple warning
felix-schwarz May 15, 2024
7d007e2
- fix #1329
felix-schwarz Apr 23, 2024
9802be1
- fix #1333 by adding a text alternative for the colored status dot
felix-schwarz Apr 29, 2024
d40c693
- fix #1340 by adding missing strings and German translation
felix-schwarz Apr 29, 2024
c94c953
- ownCloud.xcscheme: make focus debugging options available
felix-schwarz May 17, 2024
3fcff0b
- AccountController+ItemActions: add context menu action for "Disconn…
felix-schwarz May 17, 2024
2552083
- DisplayHostViewController:
felix-schwarz May 22, 2024
d3cd5d4
- ActionCell: direct keyboard focus to embedded buttons, allowing to …
felix-schwarz May 22, 2024
0b25ef2
- SortBar: add VoiceOver labels including direction to sort popup but…
felix-schwarz May 23, 2024
6f11ba8
- SegmentViewItem: add accessibilityLabelSummary accessor that compil…
felix-schwarz May 23, 2024
f1c1c10
- ActionCell:
felix-schwarz May 23, 2024
8d6dd8e
- DriveListCell: add accessibility trait to fix issue 7 in #1341
felix-schwarz May 23, 2024
123edd4
- AccountControllerCell: add accessibility label and role, fixing iss…
felix-schwarz May 23, 2024
154809d
Merge branch 'master' into fix/a11y
felix-schwarz May 27, 2024
68f84fc
- Action:
felix-schwarz May 31, 2024
6677219
- AccountControllerSpacesGridViewController: fix finding by @jesmrec …
felix-schwarz Jun 6, 2024
867c333
- remove "Drag Item" accessibility custom action for "Quick Access" s…
felix-schwarz Jun 10, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .swiftlint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,10 @@ disabled_rules:
- function_parameter_count
- comment_spacing
- unused_closure_parameter
- control_statement
- nesting
- comma
- large_tuple
custom_rules:
empty_line_after_guard_statement:
included: ".*\\.swift"
Expand Down
2 changes: 2 additions & 0 deletions PrivacyInfo.xcprivacy
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@
<string>E174.1</string>
</array>
</dict>
<dict>
<key>NSPrivacyAccessedAPIType</key>
<string>NSPrivacyAccessedAPICategoryUserDefaults</string>
<key>NSPrivacyAccessedAPITypeReasons</key>
<array>
Expand Down
Binary file added design/text-uri-list.idraw
Binary file not shown.
60 changes: 60 additions & 0 deletions doc/CONFIGURATION.json
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,10 @@
"description" : "Create folder",
"value" : "com.owncloud.action.createFolder"
},
{
"description" : "Create shortcut",
"value" : "com.owncloud.action.createShortcutFile"
},
{
"description" : "Cut",
"value" : "com.owncloud.action.cutpasteboard"
Expand Down Expand Up @@ -65,6 +69,10 @@
"description" : "Move",
"value" : "com.owncloud.action.move"
},
{
"description" : "Open shortcut",
"value" : "com.owncloud.action.openShortcutFile"
},
{
"description" : "Open in",
"value" : "com.owncloud.action.openin"
Expand Down Expand Up @@ -179,6 +187,10 @@
"description" : "Create folder",
"value" : "com.owncloud.action.createFolder"
},
{
"description" : "Create shortcut",
"value" : "com.owncloud.action.createShortcutFile"
},
{
"description" : "Cut",
"value" : "com.owncloud.action.cutpasteboard"
Expand Down Expand Up @@ -207,6 +219,10 @@
"description" : "Move",
"value" : "com.owncloud.action.move"
},
{
"description" : "Open shortcut",
"value" : "com.owncloud.action.openShortcutFile"
},
{
"description" : "Open in",
"value" : "com.owncloud.action.openin"
Expand Down Expand Up @@ -346,6 +362,38 @@
"status" : "advanced",
"type" : "string"
},
{
"autoExpansion" : "none",
"category" : "Actions",
"categoryTag" : "actions",
"classIdentifier" : "action",
"className" : "ownCloudAppShared.Action",
"defaultValue" : "all",
"description" : "Determines how the app opens shortcut files (ending in `.url`) app.",
"flatIdentifier" : "action.open-shortcut-mode",
"key" : "open-shortcut-mode",
"label" : "Open Shortcut mode",
"possibleValues" : [
{
"description" : "Open all shortcut files, targeting both links (web and other) and items.",
"value" : "all"
},
{
"description" : "Open only shortcut files that target items.",
"value" : "items-only"
},
{
"description" : "Open only shortcut files that target links (web and other).",
"value" : "links-only"
},
{
"description" : "Do not open shortcut files.",
"value" : "none"
}
],
"status" : "advanced",
"type" : "string"
},
{
"autoExpansion" : "none",
"category" : "App",
Expand Down Expand Up @@ -1758,6 +1806,10 @@
"description" : "Extension with the identifier com.owncloud.action.createFolder.",
"value" : "com.owncloud.action.createFolder"
},
{
"description" : "Extension with the identifier com.owncloud.action.createShortcutFile.",
"value" : "com.owncloud.action.createShortcutFile"
},
{
"description" : "Extension with the identifier com.owncloud.action.cutpasteboard.",
"value" : "com.owncloud.action.cutpasteboard"
Expand Down Expand Up @@ -1790,6 +1842,10 @@
"description" : "Extension with the identifier com.owncloud.action.move.",
"value" : "com.owncloud.action.move"
},
{
"description" : "Extension with the identifier com.owncloud.action.openShortcutFile.",
"value" : "com.owncloud.action.openShortcutFile"
},
{
"description" : "Extension with the identifier com.owncloud.action.openin.",
"value" : "com.owncloud.action.openin"
Expand Down Expand Up @@ -1902,6 +1958,10 @@
"description" : "Extension with the identifier org.owncloud.ql_preview.",
"value" : "org.owncloud.ql_preview"
},
{
"description" : "Extension with the identifier org.owncloud.url-shortcut.",
"value" : "org.owncloud.url-shortcut"
},
{
"description" : "Extension with the identifier org.owncloud.webview.",
"value" : "org.owncloud.webview"
Expand Down
47 changes: 47 additions & 0 deletions doc/configuration.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@ tag::actions[]
! `com.owncloud.action.createFolder`
! Create folder

! `com.owncloud.action.createShortcutFile`
! Create shortcut

! `com.owncloud.action.cutpasteboard`
! Cut

Expand All @@ -56,6 +59,9 @@ tag::actions[]
! `com.owncloud.action.move`
! Move

! `com.owncloud.action.openShortcutFile`
! Open shortcut

! `com.owncloud.action.openin`
! Open in

Expand Down Expand Up @@ -142,6 +148,9 @@ action.create-document-mode
! `com.owncloud.action.createFolder`
! Create folder

! `com.owncloud.action.createShortcutFile`
! Create shortcut

! `com.owncloud.action.cutpasteboard`
! Cut

Expand All @@ -163,6 +172,9 @@ action.create-document-mode
! `com.owncloud.action.move`
! Move

! `com.owncloud.action.openShortcutFile`
! Open shortcut

! `com.owncloud.action.openin`
! Open in

Expand Down Expand Up @@ -271,6 +283,32 @@ action.open-in-web-app-mode

|advanced `candidate`

|**Open Shortcut mode** +
+
action.open-shortcut-mode
|string
|`all`
|Determines how the app opens shortcut files (ending in `.url`) app.
[cols="1,1"]
!===
! Value
! Description
! `all`
! Open all shortcut files, targeting both links (web and other) and items.

! `items-only`
! Open only shortcut files that target items.

! `links-only`
! Open only shortcut files that target links (web and other).

! `none`
! Do not open shortcut files.

!===

|advanced `candidate`

|===
end::actions[]

Expand Down Expand Up @@ -1098,6 +1136,9 @@ tag::extensions[]
! `com.owncloud.action.createFolder`
! Extension with the identifier com.owncloud.action.createFolder.

! `com.owncloud.action.createShortcutFile`
! Extension with the identifier com.owncloud.action.createShortcutFile.

! `com.owncloud.action.cutpasteboard`
! Extension with the identifier com.owncloud.action.cutpasteboard.

Expand All @@ -1122,6 +1163,9 @@ tag::extensions[]
! `com.owncloud.action.move`
! Extension with the identifier com.owncloud.action.move.

! `com.owncloud.action.openShortcutFile`
! Extension with the identifier com.owncloud.action.openShortcutFile.

! `com.owncloud.action.openin`
! Extension with the identifier com.owncloud.action.openin.

Expand Down Expand Up @@ -1206,6 +1250,9 @@ tag::extensions[]
! `org.owncloud.ql_preview`
! Extension with the identifier org.owncloud.ql_preview.

! `org.owncloud.url-shortcut`
! Extension with the identifier org.owncloud.url-shortcut.

! `org.owncloud.webview`
! Extension with the identifier org.owncloud.webview.

Expand Down
1 change: 1 addition & 0 deletions img/filetypes-tvg/text-uri-list.tvg
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
{"viewBox":"{{0, 0}, {16, 16}}","image":"<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<!DOCTYPE svg PUBLIC \"-\/\/W3C\/\/DTD SVG 1.1\/\/EN\" \"http:\/\/www.w3.org\/Graphics\/SVG\/1.1\/DTD\/svg11.dtd\">\n<svg version=\"1.1\" xmlns=\"http:\/\/www.w3.org\/2000\/svg\" xmlns:xlink=\"http:\/\/www.w3.org\/1999\/xlink\" x=\"0\" y=\"0\" width=\"16\" height=\"16\" viewBox=\"0, 0, 16, 16\">\n <path d=\"M10.443,4.911 L10.735,4.921 L10.942,5.058 C11.186,5.302 11.186,5.698 10.942,5.942 L5.942,10.942 C5.698,11.186 5.302,11.186 5.058,10.942 C4.814,10.698 4.814,10.302 5.058,10.058 L10.058,5.058 C10.288,4.901 10.16,4.952 10.443,4.911 z\" fill=\"{{fileFillColor}}\"\/>\n <path d=\"M13.001,1 C14.105,1 15,1.895 15,2.999 L15,13.001 C15,14.105 14.105,15 13.001,15 L2.999,15 C1.895,15 1,14.105 1,13.001 L1,2.999 C1,1.895 1.895,1 2.999,1 L13.001,1 z M13.001,2.25 L2.999,2.25 C2.585,2.25 2.25,2.585 2.25,2.999 L2.25,13.001 C2.25,13.415 2.585,13.75 2.999,13.75 L13.001,13.75 C13.415,13.75 13.75,13.415 13.75,13.001 L13.75,2.999 C13.75,2.585 13.415,2.25 13.001,2.25 z\" fill=\"{{fileFillColor}}\"\/>\n <path d=\"M10.5,4.875 C10.845,4.875 11.125,5.155 11.125,5.5 C11.125,5.845 10.845,6.125 10.5,6.125 L6.5,6.125 C6.155,6.125 5.875,5.845 5.875,5.5 C5.875,5.155 6.155,4.875 6.5,4.875 L10.5,4.875 z\" fill=\"{{fileFillColor}}\"\/>\n <path d=\"M10.5,4.875 C10.845,4.875 11.125,5.155 11.125,5.5 L11.125,9.5 C11.125,9.845 10.845,10.125 10.5,10.125 C10.155,10.125 9.875,9.845 9.875,9.5 L9.875,5.5 C9.875,5.155 10.155,4.875 10.5,4.875 z\" fill=\"{{fileFillColor}}\"\/>\n<\/svg>\n","defaults":{"fileFillColor":"#969696"}}
8 changes: 8 additions & 0 deletions img/filetypes/text-uri-list.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion ownCloud Intents/GetDirectoryListingIntentHandler.swift
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ public class GetDirectoryListingIntentHandler: NSObject, GetDirectoryListingInte
if targetDirectoryQuery.sortComparator == nil {
let sort = SortMethod(rawValue: (intent.sortType.rawValue - 1)) ?? SortMethod.alphabetically

targetDirectoryQuery.sortComparator = sort.comparator(direction: SortDirection(rawValue: (intent.sortDirection.rawValue - 1)) ?? SortDirection.ascendant)
targetDirectoryQuery.sortComparator = sort.comparator(direction: SortDirection(rawValue: (intent.sortDirection.rawValue - 1)) ?? SortDirection.ascending)
}
core?.start(targetDirectoryQuery)
} else {
Expand Down
Loading
Loading