Coder Social home page Coder Social logo

Comments (22)

dlvoy avatar dlvoy commented on July 28, 2024 2

Andreas, weil ich ein bisschen verstehe, nicht jede hier ist Deutschsprachige ;)

It is obviously error with summary dialog, which does not allow scrolling on ultra-small screens. I will fix that.
In the meantime - hack mentioned above will allow to import prefs.

from androidaps.

osodebailar avatar osodebailar commented on July 28, 2024 1

Ja da muss man sich Layoutmässig was einfallen lassen.
Wenn es nur um den Import geht , dann öffne die DateI : PrefImportSummaryDialog.kt
und kommentiere die folgenden 2 Zeilen aus
Zeile 46
// val table = (innerLayout.findViewById(R.id.summary_table) as TableLayout)
und Zeile 79
// table.addView(rowLayout, idx++)

dann siehst Du keine Zusammnfassung mehr , kannst aber imortieren

from androidaps.

osodebailar avatar osodebailar commented on July 28, 2024 1

@dlvoy i have a solution that work. in the next two days i make a pull request for a review.

from androidaps.

swissalpine avatar swissalpine commented on July 28, 2024 1

I can confirm: The fix works on Unihertz Atom.

from androidaps.

osodebailar avatar osodebailar commented on July 28, 2024

Maybe the same setting as in atom ca solve the problem.
Set the fontsize in settings of jelly to 'small'.
If this does not help, you can do a fast workaround until as solution is implemented:
Search the text 'Du wirst nach dem Master-Passwort gefragt. .......' and delete the second sentence 'Mit diesem werden die importierten Einstellungen entschlüsselt' in strings.xml.
Commit the the file and rebuild master

from androidaps.

MarieT1D avatar MarieT1D commented on July 28, 2024

Thanks for the hint, but unfortunately it didn't work.
As in the third picture, there are too many listings, so shortening the phrases is not enough and changing the font size doesn't apply for me with this (or not enough) :/

from androidaps.

osodebailar avatar osodebailar commented on July 28, 2024

Than the Next workaround , Remove the unneeded Backup Filed from AAps/preferences Directory

from androidaps.

MarieT1D avatar MarieT1D commented on July 28, 2024

There is only one at AAPS/preferences (which I need to import) or which to you mean?

from androidaps.

osodebailar avatar osodebailar commented on July 28, 2024

Ja Marie , alle Backups der Einstellungen aus dem Ordner löschen die Du nicht brauchst. dann ist die Liste kurz und es sollte passen

from androidaps.

MarieT1D avatar MarieT1D commented on July 28, 2024

The list I mean contains the information of the back-up (Name, Date, phone used for the back-up, AAPS Version used for the backup, etc.) (3rd picture) so I think we are talking about a different list

from androidaps.

osodebailar avatar osodebailar commented on July 28, 2024

Ach das meinst Du. Sry. das war für mich keine Liste.
Du kannst alternativ testen den Title 'Einstellungen Importieren' zu kürzen. der Header verbraucht viel Platz.
Für eine endgültige Lösung muss das Layout überarbeitet werden.

from androidaps.

MarieT1D avatar MarieT1D commented on July 28, 2024

Already tried that, too, I changed all string variablies I could which have line breaks. The only thing I could still try is changing the size of the header.
Screenshot_20200927-162755

from androidaps.

osodebailar avatar osodebailar commented on July 28, 2024

Dafuer gibt es Google . Da geht auch English zu deutsch und und umgekehrt .
Aber ich hatte eh Marie direkt geantwortet .

from androidaps.

swissalpine avatar swissalpine commented on July 28, 2024

Yes, I deleted the German language files because the texts are too long and cumbersome for me. But it also seems to me that my font size is smaller, so I have no problems at all on the Atom. Maybe you should check the font size in the Atom settings (Einstellungen > Display > Erweitert> Schriftgröße).
Screenshot_20200927-214539

from androidaps.

MarieT1D avatar MarieT1D commented on July 28, 2024

Ja da muss man sich Layoutmässig was einfallen lassen.
Wenn es nur um den Import geht , dann öffne die DateI : PrefImportSummaryDialog.kt
und kommentiere die folgenden 2 Zeilen aus
Zeile 46
// val table = (innerLayout.findViewById(R.id.summary_table) as TableLayout)
und Zeile 79
// table.addView(rowLayout, idx++)

dann siehst Du keine Zusammnfassung mehr , kannst aber imortieren

It worked, thanks a lot!

from androidaps.

dlvoy avatar dlvoy commented on July 28, 2024

After first attempts, seems like AlertDialog.builder inner layout cannot just simple be wrapped with ScrollView.
Probably whole dialog need refactoring from system-type Dialog to custom Activity, but this will be more complex fix.

from androidaps.

dlvoy avatar dlvoy commented on July 28, 2024

Ha! I also worked on it in parallel :D here my attempt:
https://github.com/dlvoy/AndroidAPS/tree/hotfix/import-prefs-layout-uniherz

Basically it introduce new layout files for small-height & landscape devices, with fixed & scrollable table with summary:
dlvoy@d763614

Do you have device to check it? I only had tested that on Emu.
Please feel free to choose which of those two fixes is better, I can review / emu test yours as well!

from androidaps.

osodebailar avatar osodebailar commented on July 28, 2024

I will have a look tomorrow on your solution
I implemented my solution in my GUI Branch , if you want to have a look.
One of the solution for very small displays is here to set the title as marquee element , and only show the file name , not the path. The path is not interesting for a normal user.
The 3rd solution is in the end when the table in the inner element is shown. I set a nested scroll around .
You can test it here in my “new_gui_to_dev’ Branch
https://github.com/osodebailar/AndroidAPS

from androidaps.

MarieT1D avatar MarieT1D commented on July 28, 2024

I will test them, thanks for your work

from androidaps.

osodebailar avatar osodebailar commented on July 28, 2024

@MarieT1D you can test my solution, i have a pull request too of you want.

from androidaps.

dlvoy avatar dlvoy commented on July 28, 2024

Fixed by @osodebailar with #15 - so it can be re-tested from dev branch.
@MarieT1D If test on device confirms it is solved - please close issue ;)

from androidaps.

MarieT1D avatar MarieT1D commented on July 28, 2024

Yesss, it works, thanks so much for you work and sorry for the delay, I will close the issue

from androidaps.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.