sylikc / jpegview

Fork of JPEGView by David Kleiner - fast and highly configurable viewer/editor for JPEG, BMP, PNG, WEBP, TGA, GIF and TIFF images with a minimal GUI. Basic on-the-fly image processing is provided - allowing adjusting typical parameters as sharpness, color balance, rotation, perspective, contrast and local under-/overexposure.
Other
2.12k stars 124 forks source link

Suggestions & requests #101

Open yw4z opened 1 year ago

yw4z commented 1 year ago

i tried before this app many years ago i remember it was too fast and still it is, also it has really small size compared to others but its not easy to use, so i gave a one more chance 6 month ago

here is my experience

Missing features & suggestions -ico,icns,svg support -double click to toggle between fullscreen & windowed view -open with menu -my background set to black but app flashes white on opening images. you can clearly see when opening black image. its minor but blinds at night -dark mode support for context menu

Confusing features & settings -Settings not saving until i hit set as defaults button. i guess it should automatically save settings like most apps do -ExchangeXButtons=true.should be set as false by default. mouse next / previous buttons mapped wrong, next works as previous & previous for next, my mouse model logitech m720 -ShowFullScreen=auto;should be set as false by default.if image size bigger than screen resolution it will automatically opens in full screen mode instead of windowed mode -DefaultWindowRect=sticky. sticky mostly known as fit image to window, i couldnt find this setting for a while -FileDisplayOrder=LastModDate.Should be set as FileName as default or auto setting can be created. Ordering could be get from file explorer from current folder. Only my downloads folder using modification date ordering so using LastModDate as default is confusing one For creating auto setting i guess you can use shell. i can get which window has which ordering on powershell

(new-object -com Shell.Application).Windows()|%{
write-host "Ordering      : " $_.Document().SortColumns
write-host "Window Handle : " $_.HWND()
write-host "Window Path   : " $_.Document().Folder.Self.path
}

Thanks for keeping it alive

sylikc commented 1 year ago

-ico,icns,svg support I just tried opening an ICO file and it opened just fine. Can you send me a sample to test with if it's not opening for you?

-open with menu exists already, see https://github.com/sylikc/jpegview/issues/63#issuecomment-1374612548

I'll look over the other suggestions more carefully.

And, the settings thing is kinda right now really wonky, and it'll take me some time to get through all the glitches and problems.

yw4z commented 1 year ago

thanks your efforts on new release, i checked ico files opening in jpegview without problems, it shows highest resolution one

configured my open with menu thanks for tip. also i got an idea for context menu. maybe it can be fully customizable like on notepad++. that would be cool using with less items it has too many items on current status.

this one their xml template https://github.com/notepad-plus-plus/notepad-plus-plus/blob/master/PowerEditor/src/contextMenu.xml

they preferred to give ids to predefined items https://github.com/notepad-plus-plus/notepad-plus-plus/blob/master/PowerEditor/installer/nativeLang/english.xml

UserCmd and OpenWith also will be handy with this

ofc many ways to skin cat, have a nice day

sorry mistakenly closed

Final edit

i found double click to full screen possible with adding "MouseDblClk IDM_FULL_SCREEN_MODE" to KeyMap.txt file

here is ordered request list with fixed or unnecessary ones removed and added few -icns,svg,multi page pdf support -Customizable context menu (explained above) -Dark mode support for context menu -importing settings -Option for auto saving after rotate (without changing quality) -Show an info overlay when one of auto zoom mode, fit to mode, sampling mode... changes -Auto setting for FileDisplayOrder(explained in first post) -Can't navigate to next image on ico files -[minor bug] App flashes white while opening images even background set to black.You can clearly see when opening black image

these should be default values for settings, explained in first post ExchangeXButtons=false ShowFullScreen=false FileDisplayOrder=FileName

sylikc commented 1 year ago

Referencing the INI/defaults to #140 . The rest of the issues will be on the backlog. Some of them are duplicates, and others would take a lot of time, and not priority at this time.