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

Fixed initial tab marker (default-tab). #42

Merged
merged 1 commit into from Mar 24, 2015
Merged

Fixed initial tab marker (default-tab). #42

merged 1 commit into from Mar 24, 2015

Conversation

phgross
Copy link
Member

@phgross phgross commented Mar 24, 2015

If a tabbedview stores tis tab on the view itself (see https://github.com/4teamwork/opengever.core/blob/master/opengever/tabbedview/browser/personal_overview.py#L37 for an example), the initial class is written to the action itself. Therefore the default tab functionality does not work proper in this case, because the initial class is not reseted after a default-tab change.

@jone please have a look ...

If a tabbedview store his tab on the view itself (see https://github.com/4teamwork/opengever.core/blob/master/opengever/tabbedview/browser/personal_overview.py#L37 for an example), the `initial` class is written to the action itself. Therefore the default tab functionality does not work proper in this case, because the initial class is not reseted after a default-tab change.
@jone
Copy link
Member

jone commented Mar 24, 2015

👍 Looks good.

jone added a commit that referenced this pull request Mar 24, 2015
Fixed initial tab marker (default-tab).
@jone jone merged commit 2247447 into master Mar 24, 2015
@jone jone deleted the pg_fix_inital_tab branch March 24, 2015 13:49
phgross added a commit to 4teamwork/opengever.core that referenced this pull request Mar 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants