curriculum icon indicating copy to clipboard operation
curriculum copied to clipboard

<Advanced HTML and CSS>: <Inconsistent link leads to download instead of opening in browser>

Open leah205 opened this issue 1 year ago • 3 comments

Checks

Describe your suggestion

In the Project: homepage section both the tablet and mobile design files open in a new tab while the desktop file is downloaded.

Path

Ruby / Rails, Node / JS

Lesson Url

https://www.theodinproject.com/lessons/node-path-advanced-html-and-css-homepage

(Optional) Discord Name

No response

(Optional) Additional Comments

No response

leah205 avatar Sep 16 '24 00:09 leah205

Yes, i checked looks like what you are saying is right. This can be worked upon.

Rajaravi99 avatar Oct 09 '24 14:10 Rajaravi99

@Rajaravi99 , hello!

What is the correct behavior - downloading or opening in the separate tab? And I would like to work on this issue, if it's up for grabs.

Kind regards, Maria

mashazyu avatar Oct 12 '24 07:10 mashazyu

@mashazyu , Hello I agree, there should be singularity in how various links open on any webapp. According to me the correct behavior should be opening in a separate tab. But lets wait for, what the project owners have to say about this.

Rajaravi99 avatar Oct 14 '24 06:10 Rajaravi99

Thanks for reporting this. It seems the larger file size for the desktop image is what's causing our CDN to serve it as a downloadable stream instead of just an image like with the other two. This can be resolved by replacing the desktop image and link with a compressed version (smaller file size).

Acceptance criteria

This is a 2 PR process as per our instructions for adding/replacing curriculum images.

  • [ ] The desktop image is replaced with a smaller file size version (provided below).
  • [ ] The relevant link href in the project spec is replaced with the new image's Statically link.

Please comment below if you'd like to be assigned to this issue. Please do not open a PR unless you have been assigned by a maintainer.

Compressed desktop image:

portfolio desktop

mao-sz avatar Oct 30 '24 14:10 mao-sz

Hi, I am happy to work on it. You can assign it to me.

Rajaravi99 avatar Oct 30 '24 14:10 Rajaravi99