-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add sortable last-modified column to entry list view #129
Comments
This is a nice solution but it doesn't solve the root problem which is not being able to customize the view in any way. This would be a great feature to bring to the app in the future. |
Aside from the test failure and a few minor comments that PR looks good to me. |
Sub-issue of #17 |
@Thynix it looks like it's just a matter of updating that one line of that one test. I can take care of that and submit a pull request against this repository with the same diff as keepassx/keepassx#192 if that's all it takes? |
@jab416171 I'm hardly in a position of authority here but that sounds good to me. |
Oh right, maybe I should be asking @TheZ3ro and @droidmonkey. |
My thoughts on this were to wait until we redo the whole display of entries. I want it to be more extensible and less hard coded then it currently is. |
Covered by #849 |
My mistake... |
Add sortable last-modified column to entry list view
keepassx/keepassx#192
From KeePassX Pull Request
The text was updated successfully, but these errors were encountered: