Skip to content
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

Show a warning if the background image can't be loaded #6128

Open
Tracked by #9995
jdescelliers opened this issue May 22, 2020 · 2 comments
Open
Tracked by #9995

Show a warning if the background image can't be loaded #6128

jdescelliers opened this issue May 22, 2020 · 2 comments
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Help Wanted We encourage anyone to jump in on these. Issue-Task It's a feature request, but it doesn't really need a major design. Priority-3 A description (P3) Product-Terminal The new Windows Terminal.
Milestone

Comments

@jdescelliers
Copy link

Hi,

The backgroundImage parameter in settings.json has a somewhat permissive path, for example all the following examples work:

  • Windows directory separators: C:\\<username>\\Pictures\\some image.png
  • Unix directory separators: C:/<username>/Pictures/some image.png
  • A mix of both Windows/Unix directory separators: C:\\<username>\\Pictures/some image.png
  • URI using file scheme for filesystem access: file://C:/<username>/Pictures/some image.png
  • HTTP URI: https://somesite.xyz/image.jpg

Other related issue:

  • when backgroundImage points to a non existing file/URI, Windows terminal does not show a warning or error, it could be handled this way:
  • on a non existing file on the filesystem: throw an warning/error that the file for the given file path does not exist
  • HTTP URI: if the resource returns a status code other than 200 throw an warning/error that the URI is incorrect
  • if the file format is unsupported/unreadable: throw an appropriate error

Since users can both use Unix/Windows separators I thought It was the best to describe this issue before filing a documentation feedback.

Configuration:

  • Windows version: 10 - 1909 (18363.836) - 64 bits
  • Distribution method: compiled for master branch/deployed manually using Visual Studio 2019 (16.5.4)
  • Compilation settings: Release/64 bits / all submoduled needed to be checked out in order for compilation to work
  • Project compiled: CascadiaCode
  • Windows terminal version: - Slightly ahead of v1.0.1401.0 (first public release) pointing to commit: 8265d94

Terminal configuration:

  • settings.json location: C:\Users\<username>\AppData\Local\Packages\WindowsTerminalDev_8wekyb3d8bbwe\LocalState\settings.json
@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels May 22, 2020
@Rob-S
Copy link

Rob-S commented May 22, 2020

The ms-appdata URI scheme also works:
ms-appdata:///roaming/some-image.png

@zadjii-msft zadjii-msft changed the title Background image path format isn't normalized Show a warning if the background image can't be loaded Aug 5, 2020
@zadjii-msft
Copy link
Member

I'm not positive that XAML will actually give us the appropriate feedback if it fails to load an image, but we can always investigate. Thanks!

@zadjii-msft zadjii-msft added Area-Settings Issues related to settings and customizability, for console or terminal Help Wanted We encourage anyone to jump in on these. Issue-Task It's a feature request, but it doesn't really need a major design. Priority-3 A description (P3) Product-Terminal The new Windows Terminal. and removed Needs-Tag-Fix Doesn't match tag requirements labels Aug 5, 2020
@zadjii-msft zadjii-msft added this to the Terminal Backlog milestone Aug 5, 2020
@zadjii-msft zadjii-msft removed the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label Aug 12, 2020
@zadjii-msft zadjii-msft modified the milestones: Terminal Backlog, Backlog Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Help Wanted We encourage anyone to jump in on these. Issue-Task It's a feature request, but it doesn't really need a major design. Priority-3 A description (P3) Product-Terminal The new Windows Terminal.
Projects
None yet
Development

No branches or pull requests

3 participants