andrewz Posted September 12, 2023 Share Posted September 12, 2023 Hi maxmp, it's andrewz here, the perfectionist who bothers you in the last few weeks with Czech translation 🙂 In these days, I'm testing my first draft of the translation within the app. That means that I learn more and more about Poweramp – I'm visiting deep menus that I've never visited and try options I've never tried before. I noticed now (I'm talking about latest beta, I'm not sure about older builds) that touch targets of these sliders (first screenshot below, sections Background Details, Background Intensity etc. in Settings -> Background) are very small. When I want to change it with my finger, I miss it in like 50 percent of cases. Could you please enlarge these touch targets? I would also like you to ask you to increase space (by decreasing padding? or by increasing box width? or by making more lines?) for strings on the screen shown below (Main Screen -> long press on the album art or short press on the menu button). I'm especially concerned about the "Lyrics" string. In many translations (Czech: Text skladby, Polish: Tekst utworu, Danish: Sangtekster), it's much longer (+6, +6, +5 letters) and does not fit. Since the same string is also used in other places of Poweramp unrelated to specific track, making the translation shorter (and less accurate) is a no-go. Edit: I would also like to remind this discrepancy I brought up on Crowdin, since it might be rather related to code than strings (and since Crowdin does not allow embedded images). I rather did not change anything in translation and I'm awaiting your statement. Thanks and sorry for bothering you! Link to comment Share on other sites More sharing options...
andrewilley Posted September 12, 2023 Share Posted September 12, 2023 @andrewz For the "song" / "songs" wording, using "song(s)" in both instances would properly cover individual and multi-selections. Andre Link to comment Share on other sites More sharing options...
maxmp Posted September 14, 2023 Share Posted September 14, 2023 @andrewz thanks for the reports, these are really helpful. Some fixes were added to the dev build. As for menu label widths - menu uses lyrics_short label, which is specifically shorter and fits menu quite nice. Link to comment Share on other sites More sharing options...
andrewz Posted September 15, 2023 Author Share Posted September 15, 2023 20 hours ago, maxmp said: @andrewz As for menu label widths - menu uses lyrics_short label, which is specifically shorter and fits menu quite nice. If I'm correct, the same string from track menu (shown on my Moby's South Side screenshot) is also used in Settings –> Look and Feel. That is the problem. For Czech, translating "Lyrics" as just "Text" make sense in the first case, since it is clearly related to a specific track, but in the second case, it would be confusing and I would rather use "Text skladby" there. Making specific string for each case would also be solution, but app code would have to be changed. Link to comment Share on other sites More sharing options...
andrewilley Posted September 15, 2023 Share Posted September 15, 2023 Conversation moved to its own topic as it was not related to a specific build release. Link to comment Share on other sites More sharing options...
maxmp Posted September 15, 2023 Share Posted September 15, 2023 @andrewz lyrics_short is used for that menu only. There is just "lyrics" label which is used anywhere else:Â https://crowdin.com/translate/poweramp-player/6927/enus-cs?filter=basic&value=0#635052Â - this one has no length limit. Link to comment Share on other sites More sharing options...
andrewz Posted September 15, 2023 Author Share Posted September 15, 2023 9 hours ago, maxmp said: @andrewz lyrics_short is used for that menu only. There is just "lyrics" label which is used anywhere else:Â https://crowdin.com/translate/Poweramp-player/6927/enus-cs?filter=basic&value=0#635052Â - this one has no length limit. I solved it for Czech. However, I cannot fix it for other mentioned languages, since I cannot speak these. Thanks and sorry for the confusion. The last thing that needs resolution is that song/songs discrepancy (my last screenshot). Will we go andrewilley's way? Link to comment Share on other sites More sharing options...
maxmp Posted September 16, 2023 Share Posted September 16, 2023 @andrewz songs delete messaging will be improved in the next betas, thanks for reporting that. Link to comment Share on other sites More sharing options...
andrewilley Posted September 16, 2023 Share Posted September 16, 2023 @maxmp Might be worth updating the wording when removing items from playlists too. Using the word Delete for list entries is easily confused with physical file deletion. Suggest using "Remove" for the list entry, and "Delete" for files. Andre Link to comment Share on other sites More sharing options...
andrewz Posted October 1, 2023 Author Share Posted October 1, 2023 I noticed one more issue. There is a typo in Settings -> Look and Feel -> Lyrics -> Preffered Lyrics App: Walkman Lyrics Excension should be Walkman Lyrics Extension (link) Looking forward to next beta to test everything out! 🙂 Link to comment Share on other sites More sharing options...
andrewz Posted October 13, 2023 Author Share Posted October 13, 2023 (edited) It's pitty that hiding Lyrics and Menu buttons on the main screen does not increase space for track labels under the album art. 😞 Edited October 13, 2023 by andrewz Link to comment Share on other sites More sharing options...
andrewilley Posted October 13, 2023 Share Posted October 13, 2023 @andrewz That would be up to the skin you are using. The two default skins use full-width anyway as the Lyrics and Settings buttons, if enabled, are overlaid over the album art area anyway. Andre Link to comment Share on other sites More sharing options...
andrewz Posted October 14, 2023 Author Share Posted October 14, 2023 (edited) On 10/13/2023 at 12:54 PM, andrewilley said: @andrewz That would be up to the skin you are using. The two default skins use full-width anyway as the Lyrics and Settings buttons, if enabled, are overlaid over the album art area anyway. Andre I understand the state depends on selected skin and it's settings. Nevertheless, I find the current default skin imperfect in this regard. The strange thing is: when I align labels to the left ("default" instead of "centered"), labels are displayed over the entire width of the screen. I will probably stick to "default" now, although I would prefer "centered" with labels using the entire width. Edited October 14, 2023 by andrewz Link to comment Share on other sites More sharing options...
andrewz Posted December 19, 2023 Author Share Posted December 19, 2023 On 10/1/2023 at 1:09 PM, andrewz said: I noticed one more issue. There is a typo in Settings -> Look and Feel -> Lyrics -> Preffered Lyrics App: Walkman Lyrics Excension should be Walkman Lyrics Extension (link) Looking forward to next beta to test everything out! 🙂 The typo is still present two months later in beta released recently 😞 Link to comment Share on other sites More sharing options...
maxmp Posted December 19, 2023 Share Posted December 19, 2023 @andrewz thanks, fixed. Link to comment Share on other sites More sharing options...
andrewz Posted January 9 Author Share Posted January 9 (edited) Is there a particular reason for padding being so different in such similar menus? I'm talking about those lines with radio buttons. Edited January 9 by andrewz Link to comment Share on other sites More sharing options...
maxmp Posted January 9 Share Posted January 9 @andrewz thanks for the report, will be fixed. Link to comment Share on other sites More sharing options...
andrewilley Posted January 9 Share Posted January 9 Could you maybe add a few of the minor requested Re-sort options in there while you are at it? Just very simple stuff like Re-Sort By path>filename, and Re-Sort By Artist>Album>Track#. And also allow the 'Reverse' item, when ticked on its own, to reverse whatever the current order might be? Same for Playlists, which have the same extra gapping in their version of the Re-Sort menu too. Andre Link to comment Share on other sites More sharing options...
maxmp Posted January 9 Share Posted January 9 @andrewilley Reverse already behaves this way (just reverses whatever sorting is there). I added re-sorting by full path, but it may fail for some cases (streams, provider-imported tracks) which has no real paths. Link to comment Share on other sites More sharing options...
andrewilley Posted January 9 Share Posted January 9 @maxmp Excellent, I actually didn't know Reverse is now a stand-alone live action on existing content. I just tried it and it works fine now, I must have missed that one being added. As for full-path sorting, I guess it should behave in the same way that filenames already do - so anything without a valid path/filename gets added at the end of a re-sorted list in simple alphabetical order. And Track# sorting for the songs within Album groups (instead of forcing alpha by title) will definitely make a number of people happy, it's been quite a common request. And it'd fall back to alpha by title if there are no track numbers, so everyone should be happy. Andre Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now