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

Tab for remaining fields #858

Merged
merged 4 commits into from
Mar 5, 2016
Merged

Tab for remaining fields #858

merged 4 commits into from
Mar 5, 2016

Conversation

stefan-kolb
Copy link
Member

See #454.

@stefan-kolb stefan-kolb added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Feb 23, 2016
@simonharrer
Copy link
Contributor

When is this updated? Will this update on the fly if I change the config?

@stefan-kolb
Copy link
Member Author

Don't think so as the FieldEditor is somehow cached i think.

# Conflicts:
#	src/main/java/net/sf/jabref/gui/GenFieldsCustomizer.java
#	src/main/java/net/sf/jabref/gui/entryeditor/EntryEditor.java
#	src/main/resources/l10n/JabRef_da.properties
#	src/main/resources/l10n/JabRef_de.properties
#	src/main/resources/l10n/JabRef_en.properties
#	src/main/resources/l10n/JabRef_es.properties
#	src/main/resources/l10n/JabRef_fa.properties
#	src/main/resources/l10n/JabRef_fr.properties
#	src/main/resources/l10n/JabRef_in.properties
#	src/main/resources/l10n/JabRef_it.properties
#	src/main/resources/l10n/JabRef_ja.properties
#	src/main/resources/l10n/JabRef_nl.properties
#	src/main/resources/l10n/JabRef_no.properties
#	src/main/resources/l10n/JabRef_pt_BR.properties
#	src/main/resources/l10n/JabRef_ru.properties
#	src/main/resources/l10n/JabRef_tr.properties
#	src/main/resources/l10n/JabRef_vi.properties
#	src/main/resources/l10n/JabRef_zh.properties
@lenhard
Copy link
Member

lenhard commented Mar 4, 2016

What is the state here? What is missing to get this merged?

@simonharrer simonharrer removed the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Mar 4, 2016
@simonharrer
Copy link
Contributor

@stefan-kolb is working on it. I removed the label to make it clear that is currently under work.

@stefan-kolb
Copy link
Member Author

Think its ok to merge.

@stefan-kolb stefan-kolb added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Mar 5, 2016
stefan-kolb added a commit that referenced this pull request Mar 5, 2016
@stefan-kolb stefan-kolb merged commit 34345a3 into master Mar 5, 2016
@stefan-kolb stefan-kolb deleted the all-fields branch March 5, 2016 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants