-
-
Notifications
You must be signed in to change notification settings - Fork 135
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
Rework User Interface like Kiwix JS Windows #523
Comments
Let's break down the features of the reworked interface in Kiwix JS Windows. Refer to screenshot below:
That's all I can think of for now. |
Thanks a lot for this detailed inventory! Here is my point of view for each feature (open to discussion) :
I don't have a strong opinion on that.
Until we implement #292, kiwix-js is a generic ZIM reader. So I would keep the neutral icon. Following these Android guidelines, we might put in the hamburger menu the 3 sections (home, settings, about), and in the kebab menu the not-so-common actions (random, print, display/hide images, back/forward, zoom in/out etc). If we implement that with bootstrap, it might automatically decide if the screen is large enough to put these actions in the top bar or in the kebab menu. But it's just an idea.
Combining both actions on a single icon might be un-necessary if we implement what is suggested above
I think we should completely remove the bottom bar. It gives many technical and usability issues.
As the top bar would be merged with the search field, I believe it already saves enough vertical space : it would not be necessary to add another feature to hide the top bar.
I would not backport this feature as it is now. @kelson42 is working on an API (part of the OpenZIM spec) that could be used to get this ToC in a generic way : I think that's what we should implement (when it's ready)
It's worth backporting this in kiwix-js, but only if it's done in a generic way.
See above.
I found this behavior intuitive. But it only works if the icons are visible in the top bar. If they can be moved in a menu (depending on the screen size), it won't work. In any case, I think we should try to not reinvent the wheel, both for the UI ideas and for the code. We already use bootstrap and should probably carry on using it as much as possible instead of coding things ourselves (and upgrade bootstrap if it's necessary) |
Thanks for the full response, @mossroy . This all seems sensible. The only thing I'm not too keen on is removing the bottom bar entirely. For anyone who wants to browse one-handed (in app context), it's essential to have minimal controls at the bottom, where the thumb easily reaches on all mobile sizes, even when you have a very large-screen handheld device. For accessibility, we mustn't assume all users have use of both hands. I'm also not sure it's technically possible to replace back/forward buttons with swipes (which would be one option), since we often have content that overflows the screen (large-width images, for example). However, if we dedicate the bottom bar to navigation only, it can be made to disappear when reading (a couple of seconds after scroll stop), and re-appear on scroll (or navigation key press), which should remove the technical issues we've experienced with it. Or it could be made never to show if a user wishes. A compromise might be to allow the user to choose the position of a single bar, either top or bottom. However, I have my doubts that we can fit everything that needs to be on-screen (not hidden behind kebab) in a single bar. I'd have to experiment. I like the idea of hamburger top-left and kebab/overflow top-right. Though I also like an application icon, but it's not a big issue. Usability is the most important. Thank you for your ideas. I don't think I'm going to be able to do substantial work on this for a few weeks due to commitments, but it's good food for thought. I completely agree on using Bootstrap. It would be major major work to use any other framework. In the meantime, do we have enough new features for new release, before a UI change? |
We probably have enough new stuff for a 2.6 release, you're right. I just created #525 for that. Regarding the bottom bar: |
The first stage in implementing this issue is #527. |
@b16r05 commented in #265:
|
NB DevTools must be open for Ctrl-Shift-R to work consistently. |
Ok thanks you so much
…On Fri, 21 Apr 2023, 7:25 pm Jaifroid, ***@***.***> wrote:
NB DevTools must be open for Ctrl-Shift-R to work consistently.
—
Reply to this email directly, view it on GitHub
<#523 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AZRVEQTPQMMLY2LTAFHMJM3XCKGUJANCNFSM4H3BXKGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Is it right |
Looks great to me! When you're ready, please make a PR. |
Yes I will raise PR just Finished The responsive task |
I make a new PR please Check it revokNavbar |
Hey @Jaifroid , |
@Rbcoder1 Great! You are still assigned. There have been a few changes with the development process: please read CONTRIBUTING again, where everything is described. Basically, we are now using ES6 modules, and we transpile them to ES5 with a build process. This builds the production app in the You might want to open a new PR, because the old PR would need to be extensively rebased. However, feel free to reopen the old one if you prefer. |
Ok I Read The Contribution Guide And Then Start Working On.
yes I make new PR With new updates ok.
…On Thu, 20 Jul 2023, 11:40 am Jaifroid, ***@***.***> wrote:
@Rbcoder1 <https://github.com/Rbcoder1> Great! You are still assigned.
There have been a few changes with the development process: please read
CONTRIBUTING again, where everything is described. Basically, we are now
using ES6 modules, and we transpile the to ES5 with a build process. This
builds the production app in the dist directory. However, you can still
develop from the root directory of the Repo, as everything should still
work in a modern browser.
You might want to open a new PR, because the old PR would need to be
rebased. However, feel free to reopen the old one if you prefer.
—
Reply to this email directly, view it on GitHub
<#523 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AZRVEQWJRBKVQNCVCBFDPQTXRDDWTANCNFSM4H3BXKGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
i Just Start My Development As Per You Suggest Me Here I Start working on 1 PR ok |
I just need to change the text of kiwix into icon |
This issue is a combination of #267, #304 and #337 (all of which are issues concerning the UI). The aim is to backport a version of the Bootstrap UI used in Kiwix JS Windows (originally used by @sharun-s in his fork of Kiwix JS). I think this work needs to be done before any more progress can be made on #127 and #173, for example. I'm opening a dedicated issue for this, so that we can set out a road-plan of what is desired and what is not desired before starting to backport, with the aim of avoiding undertaking unnecessary work.
The text was updated successfully, but these errors were encountered: