Menno, since you touched this recently - can you make the magnifying glass icon visible if there is a partial response and token persistence is activated?

Another note: I see you replaced the input icons with linked icons (which is great!) for issue 07082 - but the icon position is wrong since then (positioning and spacing) - can you please have a look at this too? Thank you ;).

Can you explain what you mean by misplaced? In blobblueish it looks ok, maybe a little tight on the spacing, but I don't see what is wrong with the positioning.

I can see that now too ... have uploaded a jpeg of the differences in spacing before and after the changes.

Ok had another look, i think i see what you mean with position, it leaves me one question: the magnifying glass and gear icon share the same position. That makes it impossible to have both for an uncompleted repsonse I think.

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=11266

Think we can add : do the survey if Uses left are up to 0 (or allow edit answer) (in token browse)

For view response detail : why not for all respoinse, even if not closed ? Need a larger cell, or icon on :over only.

did you check out my last commit? I always add the show response icon when it is not anonymous and a token is found in the responses. It takes you to the browse response screen, filtered on the token so you can see all (partial + complete) responses for that particular token.

pfpDave, feedback please?

Just upgraded my test environment to the latest build hoping to test this quickly for you but looks like Web Auth is broken again in this new build :( Will test properly later but it isn't asking me to login, simply giving me the error 'Incorrect username and/or password!' on the login screen, refreshing the page and closing and re-opening the browser return the same result when visiting http://platform_url/admin

I have very little time at the moment so will have to park this for now.

There was a patch for 07049 . Please have a look if reverting that patch resolves your web auth issue.

Looking at the code, it does look like that's what will have broken my Web Auth ... Remote_user and Logon_user are both set for me (to the same), the issue is that the code block for logon_user will remove the 'domain\' prefix but the new code block for Remote_user doesn't ie this line:

$sUser = substr($sUser, strrpos($sUser, "\")+1);

To expand, the server variables grab my username as DOMAIN\USERID and the code needs only to match the 'USERID' part.

Dave

OK have now tested (by the way I got an error about language params 'langcode param is undefined' when I logged in, corrected by editing my profile and clicking save ... DB script required for this upgrade perhaps??) Error attached FYI.

Changes is exactly what I was after, thanks ... slight spacing issue which is incredibly minor and I wouldn't normally mention but may be of interest ... first is that the sort toggle icon is half hidden to start with and then when column expanded to view up and down arrows don't line up ... are they meant to? Images uploaded FYI.

For me though this can be closed, thanks again.

Argh can you delete that last htm file I uploaded please?

Done.