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

"Mark as read" doesn't work in IE11 #772

Closed
rnwgnr opened this issue Jan 26, 2015 · 6 comments
Closed

"Mark as read" doesn't work in IE11 #772

rnwgnr opened this issue Jan 26, 2015 · 6 comments
Assignees
Labels
Milestone

Comments

@rnwgnr
Copy link

rnwgnr commented Jan 26, 2015

Hi,

i recently switched to FreshRSS. I started with the official version 0.8.1 and updated later to github-version 0.9.4.
It looks like the "Mark as read"-button does not work when using Internet Explorer 11.
Everything is fine with Opera 26.

Has anyone come accross the same issue?

Kind regards.

@marienfressinaud
Copy link
Member

Thanks for reporting.

I guess problem comes from "Mark as read" button was in fact a link before (i.e. <a /> tag) and now it is a real button (for security reasons) but with form and formaction attributes which may be not supported by IE11.

Time to say that, I have checked on the IE dev center which says formAction is supported since IE 10. My guess is IE doesn't support formaction without a capital A but in this case it is a bug from IE (html is case insensitive).

Sadly, I have no Internet Explorer under my keyboard to test myself :(

@rnwgnr
Copy link
Author

rnwgnr commented Jan 26, 2015

Thanks for the fast reply.
I´ll try to prove your assumption this evening and report the result.

IE ist not my default browser, but unfortunately there is one environment where i can´t switch browser as i´d like to. ;)

@rnwgnr
Copy link
Author

rnwgnr commented Jan 26, 2015

Hi,

using a capital A doesn´t do anything, still nothing happening with IE11.
I´ll ask a colleague tomorrow who should be more familiar with these things than i am.

Keep you updated.

Kind regards

@Alkarex
Copy link
Member

Alkarex commented Jan 26, 2015

No, the formaction works fine in IE. The problem seems to be with the form attribute. Moving the button inside the form makes it work.
MSDN does not say anything about this attribute being writeable https://msdn.microsoft.com/en-us/library/ie/ms533768.aspx

@Alkarex Alkarex added Critical ☠️ UI 🎨 User Interfaces labels Apr 5, 2015
@Alkarex
Copy link
Member

Alkarex commented May 10, 2015

FYI, I have just tested with a preview of Microsoft Edge (next version of IE) and it is the same.

@Alkarex Alkarex modified the milestone: 1.1.2 May 16, 2015
@Alkarex Alkarex changed the title Version 0.9.4: "Mark as read" doesn't work in IE11 "Mark as read" doesn't work in IE11 Jun 1, 2015
@Alkarex Alkarex modified the milestones: 1.1.2, 1.1.3-beta Jul 5, 2015
@Alkarex Alkarex modified the milestones: 1.1.3-beta, 1.1.4-beta Aug 30, 2015
@Alkarex Alkarex modified the milestones: 1.1.4-beta, 1.3.0-beta Oct 24, 2015
@Alkarex Alkarex modified the milestones: 1.3.1-beta, 1.3.2-beta Feb 22, 2016
@Alkarex Alkarex removed this from the 1.3.2-beta milestone Jun 12, 2016
@Alkarex Alkarex modified the milestones: 1.5.1-beta, 1.3.2-beta Jun 12, 2016
@Alkarex Alkarex self-assigned this Jul 31, 2016
@Alkarex Alkarex added the Work in progress 🚧 Wait before merging label Jul 31, 2016
Alkarex added a commit to Alkarex/FreshRSS that referenced this issue Jul 31, 2016
@Alkarex
Copy link
Member

Alkarex commented Jul 31, 2016

Patched in #1188

@Alkarex Alkarex closed this as completed Jul 31, 2016
Alkarex added a commit to Alkarex/FreshRSS that referenced this issue Aug 15, 2016
Introduced by the new forms of
FreshRSS#772
FreshRSS#1188
javerous pushed a commit to javerous/FreshRSS that referenced this issue Jan 20, 2020
javerous pushed a commit to javerous/FreshRSS that referenced this issue Jan 20, 2020
Introduced by the new forms of
FreshRSS#772
FreshRSS#1188
@Alkarex Alkarex removed the Work in progress 🚧 Wait before merging label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants