aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/ui/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* ui: account for binding disappearing on detail fragmentJason A. Donenfeld2020-09-301-7/+11
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: hack gridlayoutmanager to fill columns before row if we're not scrollingJason A. Donenfeld2020-09-271-0/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we're horizontally scrolling, it makes sense to fill rows before columns. But if it all fits in one page and we don't need to scroll horizontally, it looks ridiculous. So, in this case, rearrange the tiles so that it appears to fill columns before rows. But we don't want things suddenly jumping around, so actually, keep the same ordering as rows-before-columns, but add invisible spaces after certain items, so that the fill area makes it look as though it's columns-before-rows. This winds up being much more visually pleasing. We do this by figuring out this kind of transformation: If we convert this matrix: 0 3 6 1 4 _ 2 5 _ To this one: 0 2 4 6 1 3 5 _ _ _ _ _ For a given index, how many spaces are under it? This changes depending on how many total are in a grid. Going from 3x3 to 4x3, for example, we have: count == 12, index = count == 11, index = 10 count == 10, index = 7,9 count == 9, index = 4,6,8 count == 8, index = 1,3,5,7 count == 7, index = 1,3,5,6! count == 6, index = 1,3,4!,5! count == 5, index = 1,2!,3!,4! count == 4, index = 0!,1!,2!,3! count == 3, index = 0!,1!,2! count == 2, index = 0!,1! count == 1, index = 0! count == 0, index = The '!' means two blanks below, no '!' means one blank below, and no mention means no blanks below. This commit adds code to compute such a table on the fly. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: queue up tunnel mutating on activity scope instead of fragment scopeJason A. Donenfeld2020-09-266-69/+80
| | | | | | | | | | | | Fragment scopes get cancelled when the fragment goes away, but we don't actually want to cancel an in-flight transition in that case. Also, before when the fragment would cancel, there'd be an exception, and the exception handler would call Fragment::getString, which in turn called requireContext, which caused an exception. Work around this by using the `activity ?: Application.get()` idiom to always have a context for strings and toasts. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: print proper exception trace from log viewerJason A. Donenfeld2020-09-261-1/+3
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: request intent permissions from hidden activityJason A. Donenfeld2020-09-261-2/+19
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: handle going up directories betterJason A. Donenfeld2020-09-242-25/+41
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: show volume descriptions for file pickerJason A. Donenfeld2020-09-241-4/+4
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: account for null data in callbackJason A. Donenfeld2020-09-241-0/+1
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: cleanup code after churnJason A. Donenfeld2020-09-235-220/+206
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: use system picker for API 29+Jason A. Donenfeld2020-09-231-10/+24
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: use our own file pickerJason A. Donenfeld2020-09-234-26/+265
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: escape deletion view with back buttonJason A. Donenfeld2020-09-231-0/+7
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: select first item after toggling deletion modeJason A. Donenfeld2020-09-221-0/+3
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: hook up isFocused as observable propertyJason A. Donenfeld2020-09-223-4/+11
| | | | | | | | This is kind of ridiculous, since the items own state should clearly be queryable, but it doesn't appear to be the case here, so just shuffle it around into kotlin and back. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: add text when there are no tunnelsJason A. Donenfeld2020-09-223-1/+18
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: make cards slightly smallerJason A. Donenfeld2020-09-221-1/+1
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: select the right thing on loadJason A. Donenfeld2020-09-221-2/+9
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: tweak TV layout to fit 3 rows betterHarsh Shandilya2020-09-221-6/+8
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* tv: set layout manager from XMLHarsh Shandilya2020-09-222-2/+2
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* tv: make logo almost betterJason A. Donenfeld2020-09-222-9/+21
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: anchor RV bottom to top of delete buttonHarsh Shandilya2020-09-221-1/+1
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* tv: remove useless attributeJason A. Donenfeld2020-09-221-1/+0
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: use plus instead of text for importingJason A. Donenfeld2020-09-221-3/+4
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: do themingHarsh Shandilya2020-09-227-19/+76
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* tv: account for broken TVs with no file pickerJason A. Donenfeld2020-09-222-1/+8
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: remove tiny words from tv bannerJason A. Donenfeld2020-09-221-0/+0
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: do not redisplay stats when deletingJason A. Donenfeld2020-09-221-3/+3
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: add tv bannerJason A. Donenfeld2020-09-224-0/+220
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: use proper icon for buttonJason A. Donenfeld2020-09-221-3/+6
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: hide deletion button when nothing to deleteJason A. Donenfeld2020-09-221-0/+1
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: add ugly deletion modeJason A. Donenfeld2020-09-224-10/+64
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: wire in statsJason A. Donenfeld2020-09-224-13/+60
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: wire up tunnel start/stopJason A. Donenfeld2020-09-223-10/+59
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: begin to wire up databindingsJason A. Donenfeld2020-09-223-47/+48
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: abstract out tunnel importingJason A. Donenfeld2020-09-225-267/+175
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: initial draft of Android TV supportHarsh Shandilya2020-09-226-12/+286
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* TunnelManager: catch exception in intent receiverJason A. Donenfeld2020-09-211-1/+7
| | | | | | | | | | | | | | | | | | | java.lang.IllegalStateException: at android.app.ContextImpl.startServiceCommon (ContextImpl.java:1720) at android.app.ContextImpl.startService (ContextImpl.java:1675) at android.content.ContextWrapper.startService (ContextWrapper.java:669) at com.wireguard.android.backend.GoBackend.startVpnService (GoBackend.java:4) at com.wireguard.android.backend.GoBackend.setStateInternal (GoBackend.java:4) at com.wireguard.android.backend.GoBackend.setState (GoBackend.java:2) at com.wireguard.android.model.TunnelManager$setTunnelState$2$1.invokeSuspend (TunnelManager.java:6) at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (BaseContinuationImpl.java:2) at kotlinx.coroutines.DispatchedTask.run (DispatchedTask.java:2) at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely (CoroutineScheduler.java) at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask (CoroutineScheduler.java:7) at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker (CoroutineScheduler.java:7) at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run (CoroutineScheduler.java:7) Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* TunnelEditorFragment: do not assume a contextJason A. Donenfeld2020-09-201-2/+2
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* TunnelListFragment: do not assume binding always existsJason A. Donenfeld2020-09-201-4/+4
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* TunnelDetailFragment: use kotlin coroutine for timer and rework nullabilityJason A. Donenfeld2020-09-201-43/+38
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* BaseFragment: avoid using requireContext() in permission result callbackJason A. Donenfeld2020-09-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | java.lang.IllegalStateException: at androidx.fragment.app.Fragment.requireContext (Fragment.java:17) at com.wireguard.android.fragment.BaseFragment$setTunnelStateWithPermissionsResult$1.invokeSuspend (BaseFragment.java:4) at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (BaseContinuationImpl.java:2) at kotlinx.coroutines.UndispatchedCoroutine.afterResume (UndispatchedCoroutine.java:19) at kotlinx.coroutines.AbstractCoroutine.resumeWith (AbstractCoroutine.java:13) at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (BaseContinuationImpl.java:2) at kotlinx.coroutines.UndispatchedCoroutine.afterResume (UndispatchedCoroutine.java:19) at kotlinx.coroutines.AbstractCoroutine.resumeWith (AbstractCoroutine.java:13) at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (BaseContinuationImpl.java:2) at kotlinx.coroutines.DispatchedTask.run (DispatchedTask.java:2) at android.os.Handler.handleCallback (Handler.java:790) at android.os.Handler.dispatchMessage (Handler.java:99) at android.os.Looper.loop (Looper.java:164) at android.app.ActivityThread.main (ActivityThread.java:7025) at java.lang.reflect.Method.invoke (Method.java) at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:441) at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1408) Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* AddTunnelsSheet: disable qrcode scanning if no cameraJason A. Donenfeld2020-09-201-1/+8
| | | | | | Part of the enhancements for Android TV. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: add tooling label for exclusions buttonHarsh Shandilya2020-09-201-1/+2
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* ui: add navigation hints for D-Pad and IMEHarsh Shandilya2020-09-207-0/+106
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* ui: add xhdpi banner resourceHarsh Shandilya2020-09-181-0/+0
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* SettingsActivity: account for module present but no rootJason A. Donenfeld2020-09-181-0/+9
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: move to Jetpack DataStore instead of SharedPrefsJason A. Donenfeld2020-09-188-101/+302
| | | | | | | Hopefully PreferencesPreferenceDataStore gets to go away sometime down the line. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: reformat codeJason A. Donenfeld2020-09-183-5/+2
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: retire EdgeToEdgeHarsh Shandilya2020-09-176-101/+1
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* ui: replace deprecated onActivityCreated with onViewCreatedHarsh Shandilya2020-09-171-2/+2
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>