-
-
Notifications
You must be signed in to change notification settings - Fork 376
File tooltip #3518
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
Comments
@Jack251970 By the way, it would be very useful for me (and I'm sure for others too) if the file information also included the age of the file (i.e. how many days, months and years have passed since it was last modified), as it is in the LISTARY preview. |
Sorry, you mean this plugin? https://github.com/Flow-Launcher/Flow.Launcher.Plugin.QuickLook |
@Jack251970 |
Could you please move this issue to that repository? Thanks! |
I have already opened a topic there before. |
Good! So I think we do not need to open another issue here. |
@Jack251970 |
Get it. |
@Jack251970 |
You can check here:
Basically, if you want to edit context menu, you can check IContextMenu interface. |
@Jack251970 |
The things I talked about are all related to development. (I would like to share you with this because this emhancement is very easy to implement from code side. So it is a good idea to implement this by yourself) |
Could you please do a PR for this and upload here for me the compiled file with the fix for this? |
I've been busy with some other things currently and may not be able to do this. I'll follow on with this later if I get a chance. |
@H5820121 I checked the codes and found the original codes have already set |
Could you please explain more what you mean? Please allow me to explain myself, since this is very important to me:
In Windows File Explorer and Everything, the tooltip contains this information, and why would this cause confusion in FL? On the contrary, I am a new user of FL and it caused me confusion that the tooltip in FL does not have this information, since I am used to Windows File Explorer and Everything receiving this information in the tooltip.
But what will users of the QuickLook plugin do, since then the preview panel cannot be seen. I opened a topic about this, but it doesn't seem like it will be fixed. |
@H5820121 Have you enabled Always Show Preview Panel feature? |
@H5820121 As you can see above, I have one file |
With the QL plugin this doesn't work well!
Of course, I intend to display the file modification date (and better yet its age) in the tooltip in addition to that. Thanks, I greatly appreciate your willingness to help! |
For me, I have two concerns:
@jjw24 @onesounds What do you think about this? |
I understand the necessity of this suggestion and will consider it for review.
this issue could be resolved through those means instead. This request arises because neither (1) nor (2) are currently available. In our case, the tooltip is currently used primarily to show the full file path, and changing it directly would not be ideal. Another option could be using a popup-style control that fixes and displays the information at a specific location. In any case, this proposal should not be implemented as-is. |
I am in favor of an approach that respects the user and gives him the option (in the settings) to choose what is best for him. Please do not "bury" this topic, because for myself (and I am sure for others, at least for QL plugin users) it is of great importance! |
@onesounds I wonder if you have any ideas? |
@Jack251970
By the way, please add an option to display the month by its name (MMM or MMMM). In the settings, many options for displaying the date, but this simple option is missing. |
Please wait a little. I understand your request, but we need some time to think it through. Although it may seem like a simple setting, we need to review the tooltip-related code across all plugins to ensure nothing breaks. We also need to decide how to handle plugins that currently have no tooltips. Personally, I try to avoid adding too many options. I also prefer not to add features that mainly cater to power users. We also have to think about things like:
You’re making a request based on your needs, but you’re not necessarily considering all these other factors. You can certainly say "I need this," but from my perspective, this isn't a critical issue that renders the program unusable. As I mentioned earlier, wouldn’t it be better to ask the QuickLook plugin developer to add file information display? We often receive requests asking us to turn Flow into a full file manager, but I’m fundamentally opposed to moving in that direction (and frankly, it’s not feasible either). I always try to focus on the root of a request. In that case, I saw the real issue as "improving the shortcut system," and solving that properly addressed everything. In any case, I told you we would think about it because I have actually considered before whether tooltips should display more useful information. I took the time to write this long reply because I believe your idea is worth considering. Please also understand that many of us joined this project as ordinary users, and are now working on it voluntarily. So please, have a little sympathy for us. |
Thank you for the detailed response, I would like to make just a few points:
The issue is to give this an option in the settings, and those who do not want this should not select this setting.
Unfortunately, this solution is not practical, because PDF and DOCX files, etc., often take a considerable amount of time to open their preview in QL, and it does not make sense to wait just for the file info button. And QL is based on different plugins for different file types from different developers, and then you have to open an issue with each of them with the will of each of them. What does seem to be a practical solution is to provide a separate keyboard shortcut for the FL QL plugin. I opened a issue for this in the past Flow-Launcher/Flow.Launcher.Plugin.QuickLook#7 , and unfortunately I haven't received any response so far and it doesn't look like I'll get one there.
And I'll end with thanks and appreciation for your wonderful work! |
@H5820121 Because our 1.20.0 version is about to be released, I plan to focus on fixing bugs for now. After that version is released, if there is a chance, I will consider this. |
@Jack251970 |
No problem. |
@Jack251970 |
@H5820121 Could you please check if #3631 works for you? https://ci.appveyor.com/project/JohnTheGr8/flow-launcher/builds/52177105/artifacts |
@Jack251970 Thank you very much! And I get the following error: |
Please wait to tomorrow for test |
@Jack251970 |
Those formats are following the settings in preview option. So you can change them here |
@Jack251970 |
Well, it is not supported and I am not planned to do that |
Hi, I have already uploaded the installer here: https://www.jumbomail.me/en/gallery/564B586B596B774F717133464B3342347436653838673D3D. |
@Jack251970 (Regarding the addition of the MMM month format, I was sorry to hear, maybe on second thought you could add it anyway?, it would be very helpful!) |
I get it. Will add support for that |
It will be available in Flow Preleases once it is merged. |
So please wait for others' reviews |
Thank you, I appreciate it very much! |
Can you please make the file tooltip (in the search results) display the file's properties (size, modification date, and preferably the file's age) like the Windows File Explorer tooltip and Everything.
This is especially necessary for users of QuickLook preview, who then have no way of seeing the above properties.
Thank you!
The text was updated successfully, but these errors were encountered: