Skip to content
This repository was archived by the owner on May 18, 2024. It is now read-only.

Update jquery-rails and add jquery-ui-rails gem - #151

Open
zwippie wants to merge 8 commits into
masterfrom
am-update-jquery-ui
Open

Update jquery-rails and add jquery-ui-rails gem#151
zwippie wants to merge 8 commits into
masterfrom
am-update-jquery-ui

Conversation

@zwippie

@zwippie zwippie commented Sep 21, 2016

Copy link
Copy Markdown
Contributor

No description provided.

@timraasveld

timraasveld commented Sep 23, 2016

Copy link
Copy Markdown
Contributor

gaat niet helemaal goed, voor:
image
na:
image

Ik weet niet of er ook nog andere dingen stuk gaan, misschien handig om alle plekken waar jquery-ui wordt gebruikt even handmatig te testen.

Ook in roqua, want door een conflict in jquery-rails versies moet de versie in roqua ook worden opgehoofd.

@zwippie

zwippie commented Sep 26, 2016

Copy link
Copy Markdown
Contributor Author

Ik heb de jquery-ui css en images ook een update gegeven naar de laatste versie. Dialog ziet er al iets beter uit (knoppen werken) maar de positie is nog steeds linksboven in het scherm. Geen idee waarom.

@Hampei Hampei left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMLGTM

@timraasveld

timraasveld commented Oct 6, 2016

Copy link
Copy Markdown
Contributor

Nu komt de hotkey dialog halverwege de pagina i.p.v halverwege het scherm, dus je moet bij lange vragenlijsten scrollen om wat te kunnen zien.

Ik kan zo ook geen oplossing vinden, dus lijkt me wel een redelijke middenweg om commit 2e10a94 (Center hotkeys dialog) te reverten.

LGTM als dit is opgelost

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants